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
Linting: Tutorial to setup eslint with vscode not working anymore. #27818
Comments
Thanks @QuicksVolex1 I'll bring this to the team's attention and we'll have a think/keep an eye out for the best way forward. |
Ah. So this is why my ESLINT wasnt working today.. |
After trying out various things, I stumbled upon a method that doesn't deviate much from the guide we use in the linting lesson and still is able to get us a working
I feel like it could work as a stopgap until issues with v9 and flat config are resolved. |
@TheOdinProject/javascript Opinions on how best to handle this currently? Summary:
I did briefly look at the manual set up section, which would prevent confusion with what the config file contains, but it uses ESM syntax and the instructions only indicate (Non-exhaustive) options that come to mind:
|
I've been following the repos for ESLint and their create-config CLI tool (the one given in ESLint's current "Getting Started" guide). Fortunately, a lot of things have been updated and are less confusing, and actually resemble what happens.
Therefore, assuming step 3 is not a blocker for our lesson purposes, we could amend the Linting lesson's We can then just get rid of the ESLint + Prettier part altogether, or keep only the "you might want to use both". None of the conflict stuff will be relevant for ESLint v9+. |
Seems like the lesson guide could be even more streamlined than before I agree that airbnb-base shouldn't be a blocker. The init wizard removed the options for style-guides and it doesn't seem that it'll ever be added back again, since they suggest using the When airbnb-base does add support for v9 and we decide to include it, we would still have to write up additional instructions since it wouldn't be automatically added through the init-wizard. So the instructions for setting up the basic eslint config would remain the same regardless, and there would only be specific parts related to |
We should be good to go with the changes I've highlighted above:
@gingkapls since you've been particularly active with looking into this as well, and the issue author has not checked the assignment box, would you like to be assigned to make these changes? |
Sure! I'd be happy to work on it :) |
Checks
Describe your suggestion
The tutorial for setting up eslint for vscode doesn't work, it's because eslint uses a new configuration file as default, but the extension expects the old one, so the fix for this is to switch to "Pre-release version" on extension, which took me a long time to figure out.
I also think that the setup used in the tutorial is not done in the same way as in the getting started section in the eslint website, but i'm not sure if this is a significant issue.
Path
Node / JS
Lesson Url
https://www.theodinproject.com/lessons/node-path-javascript-linting
(Optional) Discord Name
No response
(Optional) Additional Comments
No response
The text was updated successfully, but these errors were encountered: