chore(eslint-plugin): remove support for eslint 5

[thomasmichaelwallace]

Resolves #2465

As far as I understood the issue, you don't think this is a breaking change (what with v5 being very old).

Following the issue I compared against eslint 6.8 (latest) to see what messageIds weren't set then, so as to avoid regressing #2451.

[typescript-eslint]

Thanks for the PR, @thomasmichaelwallace!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!

---

🙏 Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint. As a thank you, your profile/company logo will be added to our main README which receives thousands of unique visitors per day.

[bradzacher]

Thanks for doing this!

I didn't label it as such, but it is a breaking change - any change in the supported dependencies is a breaking change (even if people should have upgraded their deps by now).
We can leave this on ice until we are ready for the next major.

[thomasmichaelwallace]

No worries - I did suspect as much. Feel free to @thomasmichaelwallace me when the time comes for v5, I'd be happy to tackle any issues updating the branch for merge.

It'd be awesome if you could label the PRs hacktoberfest-accepted in the interim. (Sorry to ask, it's annoying they put this on the maintainers; if it's a hassle, it's not a problem.)

[armano2]

we should also look at changes done in #2917 and most likely some of them can be removed, that could potentially partially fix #2279

[nx-cloud]

Nx Cloud Report

CI ran the following commands for commit 6c2803b. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Status	Command
	nx test @typescript-eslint/eslint-plugin-tslint
	nx test @typescript-eslint/eslint-plugin
	nx test @typescript-eslint/eslint-plugin-tslint
	nx test @typescript-eslint/eslint-plugin
	nx test @typescript-eslint/eslint-plugin-tslint
	nx test @typescript-eslint/eslint-plugin
	nx test @typescript-eslint/eslint-plugin-internal
	nx test @typescript-eslint/eslint-plugin-tslint
	nx test @typescript-eslint/eslint-plugin
	nx run-many --target=typecheck --all --parallel
	nx test @typescript-eslint/parser
	nx test @typescript-eslint/experimental-utils
	nx test @typescript-eslint/scope-manager
	nx test @typescript-eslint/visitor-keys
	nx test @typescript-eslint/typescript-estree
	nx run-many --target=build --all --parallel
	nx run-many --target=build --all --parallel
	nx test @typescript-eslint/parser

---

Sent with 💌 from NxCloud.

[bradzacher]

Unfortunately with eslint v8 they hid all of their utils - so we can't leverage them - so we're stuck with the changes from #2917

[bradzacher]

thanks for this! we can finally merge this in :)