Skip to content
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

Update lint and related projects #1275

Closed
samreid opened this issue Jul 11, 2022 · 6 comments
Closed

Update lint and related projects #1275

samreid opened this issue Jul 11, 2022 · 6 comments

Comments

@samreid
Copy link
Member

samreid commented Jul 11, 2022

This week we plan to have a TypeScript sprint, which will use lint for checking some of the patterns. I thought this would be a good opportunity to update our lint dependencies in advance.

Some breaking changes were noted in the change logs, but I tested the main processes and tested breaking a few lint rules and confirmed things seem to be working. Also, somehow this caught several missing semicolons which had been bothering me.
https://github.com/eslint/eslint/releases?page=1

https://stackoverflow.com/questions/69554485/eslint-error-must-use-import-to-load-es-module describes how to update the parser accordingly.

@samreid samreid self-assigned this Jul 11, 2022
samreid added a commit to phetsims/kite that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/dot that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/sun that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/scenery-phet that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/geometric-optics that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/number-play that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/ratio-and-proportion that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/tambo that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/greenhouse-effect that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/wilder that referenced this issue Jul 11, 2022
samreid added a commit to phetsims/scenery that referenced this issue Jul 11, 2022
@samreid
Copy link
Member Author

samreid commented Jul 11, 2022

Completed and I notified the slack dev-public team. I feel like a spot-check review would be good, and let's wait to see how CT does with these changes. I'll randomly choose a reviewer:

image

@chrisklus can you please take a look at your convenience?

@samreid
Copy link
Member Author

samreid commented Jul 12, 2022

@jessegreenberg and I discovered that recent changes here likely disrupt eslint in Webstorm, at least on Windows. It shows the salmon dialog at the top and says it cannot find a plugin. We continue to investigate.

@samreid
Copy link
Member Author

samreid commented Jul 12, 2022

According to https://eslint.org/docs/latest/user-guide/configuring/plugins

Plugins in the base configuration (loaded by extends setting) are relative to the derived config file.

I believe this is a new change. But from change logs it looks like this change may have been introduced in 7.0 so I'm not sure why the error was picked up now. But as @jessegreenberg and I saw, installing "@typescript-eslint/eslint-plugin": "5.30.6" in the sim repo still does not correct the problem.

@samreid
Copy link
Member Author

samreid commented Jul 12, 2022

@jessegreenberg discovered WebStorm 2021 does not handle the relative plugin directory properly, see https://youtrack.jetbrains.com/issue/WEB-54388/Extra-eslint-options-are-not-applied-when-using-Eslint-8.

So devs will need to update to WebStorm 2022.1.1 or higher after pulling (and npm update) these chipper changes.

@chrisklus
Copy link
Contributor

I reviewed the update commit in comparison with the stack overflow thread and all is looking good to me. I believe these changes have been working well for everyone on the team after updating WebStorm. Back to @samreid to close if all set here.

@chrisklus chrisklus assigned samreid and unassigned chrisklus Aug 29, 2022
@samreid
Copy link
Member Author

samreid commented Aug 29, 2022

Thanks, closing.

@samreid samreid closed this as completed Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants