-
Notifications
You must be signed in to change notification settings - Fork 81
Modernize ESLint plugin configs while maintaining backward compatibility #2924
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
base: master
Are you sure you want to change the base?
Conversation
|
@turadg 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review. |
Co-authored-by: turadg <[email protected]>
Co-authored-by: turadg <[email protected]>
Co-authored-by: turadg <[email protected]>
|
@copilot don't make a new |
Co-authored-by: turadg <[email protected]>
I understand you want me to convert the configs in However, ESLint v9 requires flat config files and doesn't support the legacy
Let me know your preference so I can implement the correct approach. |
…nt v9 upgrade path Co-authored-by: turadg <[email protected]>
Co-authored-by: turadg <[email protected]>
This PR modernizes the ESLint configuration structure in the
@endo/eslint-pluginpackage while maintaining backward compatibility with existing package configurations.Key Changes
ESLint Plugin Config Modernization
packages/eslint-plugin/lib/configs/to use modern ESLint patternseslintConfigsections in package.json filesConfiguration Files Updated
recommended.js- Core SES globals and fundamental rulesstyle.js- Airbnb base + JSDoc + Prettier integration with TypeScript overridesimports.js- ES module import/export rulesstrict.js- Combined strict configurationses.js- SES-specific restricted globals with test file exceptionsinternal.js- Complex TypeScript configuration with dynamic rule handlingdaemon.js- Deprecated configuration (maintained for compatibility)recommended-requiring-type-checking.js- TypeScript-aware rule extensionsCompatibility Preserved
eslintConfigsections in package.json files continue to work unchangedThe modernized configurations provide a foundation for future ESLint upgrades while ensuring zero disruption to the current development workflow.
Addresses #2682.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.