ember-es6-class/no-object-extend rule

[hmajoros]

checkup/packages/checkup-plugin-ember-octane/src/utils/task-configs.ts

Line 12 in 8f88c8b

rules: [

any reason this chunk isn't using the no-object-extend rule defined here?
https://github.com/scalvert/eslint-plugin-ember-es6-class

this is what our team used to track our native class migration

[scalvert]

Hey @hmajoros, thanks for the issue, and this is a good question.

There's no real reason - we could likely add it to the rest of the rules to ensure we're covering all our bases.

Thanks for the suggestion!

[lbdm44]

@scalvert is there a reason we should be using no-extend-object over the combo of the standard Ember ember/no-classic-classes and ember/no-mixins? Seems like the combination of the two cover the enforcement of ES6 Classes and removing .extend which is only used for mixins at this point.

The only possible corner case I can see is that engineers are using ES6 classes with .extend outside of mixins, but that seems irresponsible considering they should just be using ES6 class extension.