-
-
Notifications
You must be signed in to change notification settings - Fork 488
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
VSCode extension: default oxc_language_server.configPath
seems wrong
#5802
Comments
@DonIsaac I can change default value to Looks like good first issue for me) How do you think about it? |
…o .oxlintrc.json closes oxc-project#5802
No-work-required eslint migration is a large goal for us. I'm down with checking for an As a note: a lot of our users use |
@DonIsaac OK would be good to have checking for .oxlintrc.json too. So I close my pull request |
Hi 👋
While trying to move part of my linting config from eslint to oxlint, I did not understand why removing the
no-console
rule from.eslintrc
& adding it to.oxlintrc.json
did not trigger an error in my VSCode editor while it was triggering it properly running the oxlint CLI.After a short investigating, I discovered that the default config path used by the OXC VSCode language server was
.eslintrc
.It means that the oxlint language server was using my legacy eslint config instead of the new appropriate one. Should we change the default config path to be the filename documented here or at least explain that the configuration file used can be changed using VSCode
settings.json
file? Many thanks!The text was updated successfully, but these errors were encountered: