Duplicate declarations about ember-native-class-codemod HOT 7 CLOSED

In the pods/MU style, the information is one level up, the folder name that contains the file:

/MLDP.WebApp/mldp/account/action/controller.js:

Should output:

export default ActionController extends Controller {}

I think folks may want this to be even less generic, like AccountActionController, but I don't think that's something we can automate, so probably just the single enclosing folder is the best thing to add to the class name.

from ember-native-class-codemod.

Ah, I don't think we handled pod/MU style declarations of names, @ssutar we definitely should add this functionality (also PRs definitely welcome! 😄)

from ember-native-class-codemod.

I'd contribute if I even understood how this thing worked. Where is the code? Haha!

from ember-native-class-codemod.

Looks like the code is here: https://github.com/scalvert/ember-es6-class-codemod/blob/master/transforms/helpers/parse-helper.js#L477

Do we have pods code anywhere? Will the testing layout need changes like mimicking pods?

from ember-native-class-codemod.

I don’t believe any pods code exists yet, and we definitely will need new tests/fixtures that mimic the pods structure. I think the new tests should be pretty straightforward, basically an input directory and output directory, though I’m not sure if our tests handle directories yet

from ember-native-class-codemod.

@pzuraq Isn't this same as #49 where we would append the class type. So this would become

export default class ControllerController extends Controller

which is not a good name but is valid

from ember-native-class-codemod.

This should be fixed on master, let us know if you're still having issues!

from ember-native-class-codemod.