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 linting #624

Closed
goosewobbler opened this issue Jun 14, 2024 · 1 comment · Fixed by #749
Closed

Update linting #624

goosewobbler opened this issue Jun 14, 2024 · 1 comment · Fixed by #749
Labels
Milestone

Comments

@goosewobbler
Copy link
Member

goosewobbler commented Jun 14, 2024

We need to address the issue fixed by #578.

Updating the linting to use un-ts/eslint-plugin-import-x@master/docs/rules/no-extraneous-dependencies.md should cover it.

Additionally, ESLint v9 has been released and once the new version of @typescript-eslint is out we should be good to upgrade. Prior art: #598

Alternative: Biome (would replace Prettier and most ESLint plugins)
https://biomejs.dev/linter/rules/no-undeclared-dependencies/

@goosewobbler goosewobbler added this to the 6.x.y milestone Jun 14, 2024
@goosewobbler goosewobbler modified the milestones: 6.x.y, 7.x.y Jul 9, 2024
@goosewobbler goosewobbler modified the milestones: 7.x.y, 7.0.0 Aug 1, 2024
@goosewobbler goosewobbler modified the milestones: 7.0.0, 7.x.y Aug 9, 2024
@goosewobbler
Copy link
Member Author

goosewobbler commented Aug 22, 2024

Went with eslint-plugin-import-x, will switch to Biome + ESLint at some point when Biome is a bit more established.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant