Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 13, 2025

This PR migrates the ESLint configuration from the legacy .eslintrc.js format to the new flat configuration format required by ESLint v9.33.0.

Changes Made

ESLint Configuration Migration

  • Removed .eslintrc.js (legacy format)
  • Added eslint.config.js with ESLint v9 flat config format
  • Migrated all custom rules from the original configuration:
    • Semi-colon enforcement
    • Single quote preference
    • Comma-dangle disabled
    • Custom spacing and formatting rules
  • Added proper Node.js globals configuration
  • Configured separate rules for test files with Mocha globals

Dependency Updates

  • Updated ESLint from 4.9.0 to ^9.33.0
  • Removed obsolete ESLint plugins that are no longer needed with the new configuration format

Workflow Updates

  • Updated publish-artificats.yml workflow to use Node.js 20.x (from 10.15.3) for ESLint v9 compatibility
  • Updated GitHub Actions versions for better security and compatibility

Validation

  • All existing tests continue to pass
  • Code coverage maintained at 96%+
  • Linting works correctly with the new configuration
  • No changes to actual code formatting or linting behavior

The migration resolves the error encountered when dependabot attempts to upgrade ESLint:

ESLint: 9.33.0
ESLint couldn't find an eslint.config.(js|mjs|cjs) file.

Fixes #25.


💡 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.

Co-authored-by: bijujoseph <3535395+bijujoseph@users.noreply.github.com>
Copilot AI changed the title [WIP] ESLint configuration Migrate ESLint configuration to v9 flat config format Aug 13, 2025
Copilot AI requested a review from bijujoseph August 13, 2025 14:05
Copilot finished work on behalf of bijujoseph August 13, 2025 14:05
@github-actions
Copy link

File Coverage Lines Branches
All files 96% 96% 96%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 38b20f2

@bijujoseph bijujoseph marked this pull request as ready for review August 13, 2025 15:24
@bijujoseph bijujoseph merged commit 90bf9c1 into master Aug 13, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ESLint configuration

2 participants