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/json'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

N/A

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
@eslintbot eslintbot added this to Triage 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:16
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 a3c26ca into main Dec 2, 2025
24 checks passed
@mdjermanovic mdjermanovic deleted the fix-add-missing-name-property branch December 2, 2025 12:39
@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