update to eslint 9

[dynst]

https://eslint.org/docs/latest/rules/no-unused-vars#caughterrors

The only noticeable change is that the no-unused-vars lint now warns about unused errors in catch (e) statements by default.

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	neostandard@​0.12.2
	eslint@​8.47.0 ⏵ 9.39.1	+1			+45

View full report

[UlisesGascon]

LGTM! It is a great contribution. Thanks @dynst for helping the project ❤️

I will ask @wesleytodd for an additional review just in case we have a different plan for eslint, but AFAIK we are good to go 👍

[Phillip9587]

@UlisesGascon There is an open discussion expressjs/discussions#327

[bjohansebas]

I definitely prefer that we use neostandard rather than having to look up which rules to apply

[shivarm]

@UlisesGascon But this PR also contain changes related modern js practice like var to const... did you guys decided to migrate finally..?

[ctcpip]

I don't think we would accept this PR in its current state -- it's not just updating eslint, it's making a large number of code changes.

Also, we still have not yet come to an agreement on the future of linting for Express and related projects.