Skip to content

Upgrade ESLint to v10#633

Merged
mao-sz merged 3 commits intoTheOdinProject:mainfrom
mao-sz:upgrade-eslint
Feb 28, 2026
Merged

Upgrade ESLint to v10#633
mao-sz merged 3 commits intoTheOdinProject:mainfrom
mao-sz:upgrade-eslint

Conversation

@mao-sz
Copy link
Contributor

@mao-sz mao-sz commented Feb 28, 2026

Because

We currently have a barebones ESLint config with several plugins we don't even use. There's limited use for ESLint in this repo, but I don't think it'll hurt to keep it around for all the stuff excluding the actual exercise JS files learners edit. Might as well upgrade to the latest ESLint with the default rule set at the same time.

This PR

  • Upgrades ESLint to v10, including config file formats
  • Removes unused ESLint plugins
  • Adds lint workflow for PRs

Issue

Closes #XXXXX

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the location of change: brief description of change format, e.g. 01_helloWorld: Update test cases
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If this PR includes any changes that affect the solution of an exercise, I've also updated the solution in the /solutions folder

airbnb-base and plugin-import weren't even being used, and we don't
really need them for this repo's scope anyway, so they were removed
Copy link
Member

@zachmmeyer zachmmeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@mao-sz mao-sz merged commit b5db863 into TheOdinProject:main Feb 28, 2026
2 checks passed
@mao-sz mao-sz deleted the upgrade-eslint branch February 28, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants