Skip to content

Conversation

@lumirlumir
Copy link
Member

@lumirlumir lumirlumir commented Dec 2, 2025

Prerequisites checklist

What is the purpose of this pull request?

In this PR, I added the missing name property to the recommended config.

The @eslint/markdown and @eslint/js plugins now both include a name property in their recommended or all configurations, but it's missing from @eslint/css's recommended configuration.

References:

Therefore, I added the name property to keep core ESLint configurations consistent.

What changes did you make? (Give an overview)

In this PR, I added the missing name property to the recommended config.

Related Issues

Ref: eslint/json#189

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

N/A

@eslint-github-bot eslint-github-bot bot added the bug Something isn't working label Dec 2, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Dec 2, 2025
@lumirlumir lumirlumir marked this pull request as ready for review December 2, 2025 09:20
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 f115c3f into main Dec 2, 2025
22 checks passed
@nzakas nzakas deleted the fix-add-missing-name-property-to-recommended-config branch December 2, 2025 15:05
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants