Skip to content

Conversation

@lumirlumir
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request?

Hi,

In this PR, I've updated the eslint dependencies to the latest versions.

The i flag was detected by the newly added eslint-plugin-regexp dependency, so I removed it. Since there are no alphabetic characters in the regex, I believe the behavior should remain the same.

What changes did you make? (Give an overview)

Related Issues

Is there anything you'd like reviewers to focus on?

@eslintbot eslintbot added this to Triage Aug 25, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Aug 25, 2025
"configFilename": "eslint.config.mjs",
"devDependencies": [
"eslint@>=9.27.0",
"eslint@>=9.33.0",
Copy link
Member Author

Choose a reason for hiding this comment

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

This line was updated due to a CI failure, as the upstream eslint-config-xo has updated its devDependencies to eslint@>=9.33.0.

Ref: #186

Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@nzakas nzakas merged commit 7c247a6 into main Aug 25, 2025
11 checks passed
@nzakas nzakas deleted the refactor-update-eslint-dependencies branch August 25, 2025 14:43
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants