-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Setting Up A React Environment: Remove mention of eslint-config-prettier #28138
Comments
@MaoShizhong When looking into this issue, I saw that the Vite template does not use ESLint v9, so I thought I should ask about your opinion on this. Since some learners might still be using Prettier alongside ESLint, they won't have a problem using them in vanilla JavaScript. But the problem arises when learners want to use Prettier with ESLint v8 and below, when there might be conflicting rules between Prettier and ESLint. Wouldn't it be better to include the mention of eslint-config-prettier as a new package that's recommended because problems might arise when using Prettier with the ESLint version Vite uses, and then when Vite updates their template to use ESLint v9 we should change this section again? The alternative being remove the whole Keeping it clean section so we don't have to worry when Vite will update their template to use ESLint v9 if at all. |
@T0nci it still won't be an issue for v8 and below, since the base ruleset still doesn't include the deprecated style rules. Having said that, I'm exploring having the Linting lesson's bit tweaked to use the v8.57 docs and a note about v8.57 and v9+ compatibility. Plugins like airbnb and airbnb-base are not yet compatible with v9 and look to be blocked for a while yet. They also actually turn on some style rules that plugin-prettier will need to turn off. So it's not to do with eslint anymore but technically some other plugins. You'll have ti give me a bit for this since I'm down and out with Covid |
@MaoShizhong Please, take your time to rest up! This issue won't go anywhere, and like you always say we're all volunteers. Hope you feel better soon! |
@T0nci I've been thinking about this some more and I think I still want the
Therefore for most people, I have some specific changes I want to update the Linting lesson with in regards to flat config/v9 support, as well as |
@MaoShizhong Yes, of course! About the removal, should I remove the whole section or mention Prettier in place of eslint-config-prettier to remind the user of Prettier if they haven't been using it up to this point? |
I'd just remove the entire "keeping it clean" section. It's clear it ships with eslint. If people have or haven't been using prettier so far, they'll continue as such. Seems a bit odd to mention it there, especially when eslint-config-prettier is technically only relevant for specific scenarios (i.e. not relevant for the default setup). Would definitely be strange to go into the technicalities there (that's all covered in the Linting lesson with recent changes now) |
Checks
Describe your suggestion
Since eslint-config-prettier isn't mentioned in the Linting lesson in the JavaScript course, the mention of eslint-config-prettier should also be removed from this lesson. I opted to opening an issue rather than Improve on GitHub since linting errors couldn't be fixed through GitHub if there are any.
The mention is located in this section: https://www.theodinproject.com/lessons/node-path-react-new-setting-up-a-react-environment#keeping-it-clean
Path
Node / JS
Lesson Url
https://www.theodinproject.com/lessons/node-path-react-new-setting-up-a-react-environment
(Optional) Discord Name
No response
(Optional) Additional Comments
No response
The text was updated successfully, but these errors were encountered: