-
Notifications
You must be signed in to change notification settings - Fork 28.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Eslint 9 migration #230345
base: main
Are you sure you want to change the base?
Eslint 9 migration #230345
Conversation
For #230339 Starts migrating to eslint 9. Everything runs but it produces a number of errors
These seem to have getting pulled in implicitly before
@mjbvz Pushed a change to use |
However use a different filename to avoid a warning
Checks look good. Build script pass and I tested introducing a few errors and confirming they were reported. Planning to merge but please keep an eye out for any regressions or cases where our custom rules no longer apply |
cc @jrieken. Need your review as code owner as I touched |
files: [ | ||
'src/vs/workbench/contrib/terminal/**/*.ts' | ||
], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add 'src/vs/workbench/contrib/terminalContrib/**/*.ts'
here too? You'll also need to delete it at https://github.com/microsoft/vscode/blob/main/src/vs/workbench/contrib/terminalContrib/.eslintrc.json
Fixes #230339
Migrates us to eslint 9 and a flat config
TODO:
eslintrc
filesFollow ups:
schema: false