-
Notifications
You must be signed in to change notification settings - Fork 52
Fix/eslint flat config plugins #1252
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…x-a11y/github flat-configs
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes ESLint flat configuration by updating plugin dependencies and reorganizing plugin registration to avoid conflicts and ensure proper flat config compatibility.
- Updated ESLint and plugin dependencies to newer versions
- Restructured ESLint configuration to register plugins upfront and merge rules separately
- Added programmatic ESLint runner script for better integration
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
package.json | Updated ESLint and related plugin versions to newer flat config compatible versions |
eslint.config.mjs | Restructured configuration to properly register plugins and merge rules for flat config compatibility |
scripts/run-eslint.mjs | New programmatic ESLint runner that generates structured JSON output |
scripts/eslint-output.json | Generated ESLint results showing successful linting with only unused disable directive warnings |
.github/workflows/*.yml | Added dependency change conditions to trigger workflows when dependencies are updated |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
Summary
List of notable changes:
What should reviewers focus on?
Steps to test: