Skip to content

ci: use Node 20 for lint job#59

Merged
szegedi merged 1 commit into
mainfrom
fix/lint-node-version
Jun 9, 2026
Merged

ci: use Node 20 for lint job#59
szegedi merged 1 commit into
mainfrom
fix/lint-node-version

Conversation

@szegedi

@szegedi szegedi commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

ESLint 10 and @eslint/js 10 use util.styleText which requires Node >= 20 (added in Node 20.12.0). The lint job was still pinned to Node 18, causing failures in Dependabot PRs that bump those packages.

ESLint 10 and @eslint/js 10 use util.styleText which requires Node >= 20.
The test matrix already dropped Node 18 so this brings lint in line.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@szegedi szegedi requested review from a team as code owners June 9, 2026 08:51
@szegedi szegedi enabled auto-merge (squash) June 9, 2026 08:51
@szegedi szegedi merged commit 819411d into main Jun 9, 2026
11 checks passed
@szegedi szegedi deleted the fix/lint-node-version branch June 9, 2026 12:54
szegedi added a commit that referenced this pull request Jun 10, 2026
ESLint 10 and @eslint/js 10 use util.styleText which requires Node >= 20.
The test matrix already dropped Node 18 so this brings lint in line.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@szegedi szegedi mentioned this pull request Jun 10, 2026
szegedi added a commit that referenced this pull request Jun 16, 2026
ESLint 10 and @eslint/js 10 use util.styleText which requires Node >= 20.
The test matrix already dropped Node 18 so this brings lint in line.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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