Skip to content

feat: update & cleanup shared eslint/ts/prettier configurations #4096

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

Closed
wants to merge 13 commits into from

Conversation

IMB11
Copy link
Member

@IMB11 IMB11 commented Jul 31, 2025

  • Creates a new @modrinth/tooling-config package that contains all the shared configuration.
  • Migrates all eslint, prettier and typescript configs to use the shared configuration.
  • Updated to eslint 9
  • Fixes lint issues introduced.

Yes, it will be painful to rebase any existing PRs onto the main branch after this is merged, but for the sake of consistency & improved linting rules in the long run it's probably worth the brief pain.

@IMB11 IMB11 force-pushed the cal/dev-136 branch 2 times, most recently from 6cf7b07 to 29ac588 Compare August 1, 2025 13:47
@IMB11 IMB11 requested a review from Prospector August 1, 2025 14:18
@IMB11 IMB11 marked this pull request as ready for review August 1, 2025 14:19
@IMB11 IMB11 added enhancement New feature or request DevEx Improvements to developer experience labels Aug 1, 2025
@IMB11
Copy link
Member Author

IMB11 commented Aug 1, 2025

Prerequesite for #4080

@IMB11 IMB11 force-pushed the cal/dev-136 branch 3 times, most recently from a13da3e to 0932282 Compare August 9, 2025 21:57
@IMB11 IMB11 force-pushed the cal/dev-136 branch 3 times, most recently from 71c9597 to 455bc3c Compare August 13, 2025 21:50
@IMB11
Copy link
Member Author

IMB11 commented Aug 13, 2025

Reapplying, to complex to fix conflicts.

@IMB11 IMB11 closed this Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DevEx Improvements to developer experience enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

1 participant