Skip to content

Conversation

Pixel998
Copy link
Contributor

@Pixel998 Pixel998 commented Aug 5, 2025

Prerequisites checklist

What is the purpose of this pull request?

This PR updates eslint-config-eslint to the latest version and ensures the codebase adheres to the updated linting rules.

What changes did you make? (Give an overview)

  • Updated eslint-config-eslint to the latest version.
  • Fixed all lint errors reported by the updated configuration.

Note: This PR is pending the publication of eslint-config-eslint v12.

Related Issues

eslint/eslint#19945

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

@Pixel998 Pixel998 marked this pull request as draft August 5, 2025 07:25
@Pixel998 Pixel998 marked this pull request as ready for review August 17, 2025 03:20
@lumirlumir
Copy link
Member

eslint-config-eslint v12 have been released, so I think it's fine to move on.

@lumirlumir lumirlumir moved this to Needs Triage in Triage Aug 23, 2025
@mdjermanovic mdjermanovic changed the title chore: update eslint-config-eslint refactor: update eslint-config-eslint Aug 24, 2025
@mdjermanovic mdjermanovic moved this from Needs Triage to Implementing in Triage Aug 24, 2025
@mdjermanovic
Copy link
Member

I changed the tag to refactor because this modifies published code.

@mdjermanovic
Copy link
Member

@Pixel998 can you please sync with the main branch again? I'm getting more lint errors locally, specifically with the htmlTagPattern regex that has been added in the meantime in 5e6e94e.

Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

LGTM. Would like @mdjermanovic to verify before merging.

@lumirlumir lumirlumir moved this from Implementing to Second Review Needed in Triage Aug 25, 2025
Copy link
Member

@mdjermanovic mdjermanovic 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!

@mdjermanovic mdjermanovic merged commit c790317 into eslint:main Aug 25, 2025
23 checks passed
@github-project-automation github-project-automation bot moved this from Second Review Needed 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
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

3 participants