Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Use ESLint 9.x#8845

Merged
DNin01 merged 2 commits intoScratchAddons:masterfrom
DNin01:eslint-9
Feb 18, 2026
Merged

Use ESLint 9.x#8845
DNin01 merged 2 commits intoScratchAddons:masterfrom
DNin01:eslint-9

Conversation

@DNin01
Copy link
Member

@DNin01 DNin01 commented Feb 17, 2026

Resolves

@Samq64#8836 (comment)

It might be a good idea to pin the ESLint version to 10.x and update it on our own schedule.

Changes

Uses globals version ^17.3.0 and @eslint/js version ^9.39.2.

Reason for changes

Major ESLint updates may have breaking changes. ESLint 10 added new rule recommendations which caused some checks to fail.

Related

#8836 is ready for when we update to ESLint 10.

@DNin01 DNin01 requested a review from Hans5958 as a code owner February 17, 2026 20:30
@DNin01 DNin01 added scope: ci Related to the continuous integration/GitHub Actions on this issue scope: development Related to developing, documentation, DevEx, etc. labels Feb 17, 2026
@DNin01 DNin01 changed the title Specify major versions for globals and @eslint/js Use ESLint 9.x Feb 17, 2026
@DNin01
Copy link
Member Author

DNin01 commented Feb 17, 2026

We may update to version 10 once it has been out for a while.

Copy link
Member

@Hans5958 Hans5958 left a comment

Choose a reason for hiding this comment

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

LGTM (esp. when we don't have package.json)

@DNin01 DNin01 merged commit b66574f into ScratchAddons:master Feb 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: ci Related to the continuous integration/GitHub Actions on this issue scope: development Related to developing, documentation, DevEx, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants