Bryan Mishkin

@ef4 sounds good, I will remove that from the plan. Would you like to edit the [require-fetch-import](https://github.com/ember-cli/eslint-plugin-ember/blob/master/docs/rules/require-fetch-import.md) rule doc to explain these caveats? That will help make it more clear...

CC: @Turbo87

Yes, this is a false positive because it should be fine to use Ember array classes ([EmberArray](https://api.emberjs.com/ember/release/classes/EmberArray), [MutableArray](https://api.emberjs.com/ember/release/classes/MutableArray), [ArrayProxy](https://api.emberjs.com/ember/release/classes/ArrayProxy), fragmentArray from ember-data, others?) directly without relying on the prototype extensions....

Could you explain what exactly you want the lint rule to do? Lint rules know what the current file path is if that helps.

This is a good edge case that we could try fixing. It might be tricky to statically detect this usage of the service, although it should be possible. I'm not...

This is related to the [ember/no-side-effects](https://github.com/ember-cli/eslint-plugin-ember/blob/master/docs/rules/no-side-effects.md) rule. My company has a custom rule for disallowing event tracking inside computer properties that also sounds related. Haven't open sourced it because it...

I wonder if it's worth creating a separate rule `no-self-referencing-computed-properties`?

I actually think the new rule `no-self-referencing-computed-properties` should catch both macro and non-macro computed properties. No reason to limit it to just one computed property form. This new rule would...

I opened a related issue a while ago about disallowing `reads` / `alias` / `oneWay`: #686

Hmm. Allowing caching like this sounds reasonable. But we do still want to disallow other usages of assignment (as that's the entire point of the rule, and I have seen...