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?

@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Aug 5, 2025
@Pixel998 Pixel998 marked this pull request as draft August 5, 2025 07:22
@github-actions
Copy link
Contributor

Hi everyone, it looks like we lost track of this pull request. Please review and see what the next steps are. This pull request will auto-close in 7 days without an update.

@github-actions github-actions bot added the Stale label Aug 15, 2025
@github-actions github-actions bot removed the Stale label Aug 17, 2025
@Pixel998 Pixel998 marked this pull request as ready for review August 17, 2025 23:35
@nzakas nzakas moved this from Needs Triage to Implementing in Triage Aug 19, 2025
@lumirlumir
Copy link
Member

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

@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Aug 23, 2025
@mdjermanovic mdjermanovic changed the title chore: update eslint-config-eslint refactor: update eslint-config-eslint Aug 24, 2025
lumirlumir
lumirlumir previously approved these changes Aug 25, 2025
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.

@nzakas nzakas 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 5af3a5a into eslint:main Aug 27, 2025
22 checks passed
@github-project-automation github-project-automation bot moved this from Second Review Needed to Complete in Triage Aug 27, 2025
@Pixel998 Pixel998 deleted the update-eslint-config-eslint branch September 6, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion chore contributor pool

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants