eslint v9 support

[Techn1x]

There are aspects of this plugin that don't support eslint v9

TypeError: context.getAncestors is not a function
from this line

eslint-plugin-ember/lib/rules/require-return-from-computed.js

Lines 69 to 70 in 7e6bca9

	context
	.getAncestors()

context.getAncestors() is replaced in v9
https://eslint.org/blog/2023/09/preparing-custom-rules-eslint-v9/#context.getancestors()

That's just the one I found, there is potentially other things that need to change for eslint v9

[LucasHill]

Should have this resolved in : #2153

[LucasHill]

#2153 was merged but there is at least one (and possibly more) deprecations. However the changes did unblock using the eslint/compat library to bring this plugin up to v9 compatibility. The usage in your eslint.config.mjs that's working for me is...

import { fixupConfigRules } from '@eslint/compat';
import emberRecommended from 'eslint-plugin-ember/configs/recommended';
const fixedEmberRecommended = fixupConfigRules(emberRecommended);
export default [
...yourOtherRules,
...fixedEmberRecommended
]

I'd still like to fix everything else to make this plugin work natively in v9 but if someone beats me to it go ahead.

[LucasHill]

@NullVoxPopuli would it make sense to do a release to enable the above workaround until full support is in?

[NullVoxPopuli]

perhaps!

I'd like to set up release-plan on this repo so releasing is less of a process (i.e.: can do it from the phone) -- but would need to discuss with @bmish <3

[bmish]

I'm in favor of more automation for the release process, probably something like release-please in eslint/generator-eslint#175. Contributions welcome if someone wants to improve it. For now, we have a command mentioned here: https://github.com/ember-cli/eslint-plugin-ember/blob/master/RELEASE.md

[LucasHill]

I hope this will be resolved with my second attempt here:
#2159

[JoshuaKGoldberg]

Was this resolved by #2159 with a fix released in 12.2.0?

[LucasHill]

@JoshuaKGoldberg i am using the plugin on 12.2.0 on my large ember codebase with eslint 9 but i'm not sure what the done criteria are for this, such as getting the test suite running against eslint 9.

[JoshuaKGoldberg]

That sounds like support to me 😄. But I'm not an Ember user so I'm not confident. If this issue gets closed by a maintainer then I'll just go ahead and mark a ✅ on eslint/eslint#18093.

[Turbo87]

seems to work fine on ESLint 9 now with the latest release. thanks everyone! :)

[Techn1x]

It does indeed seem to work 🎉

If anyone here's having dependency issues with ember-eslint-parser & eslint v9, this PR might be relevant to you
ember-tooling/ember-eslint-parser#95