Skip to content

Conversation

@amazon-q-developer
Copy link

ESLint Configuration Update

This PR updates the ESLint configuration to use @antfu/eslint-config instead of the previously used @shopify/eslint-plugin. This represents a significant modernization of our ESLint setup with several key improvements:

Major Changes

  • Migrated to ESLint's new flat config format
  • Updated ESLint plugin to use @antfu/eslint-config as the base configuration
  • Enhanced TypeScript support and modern JavaScript/TypeScript development defaults
  • Revised documentation to reflect the new configuration approach
  • Updated configuration initialization templates and CLI tools

Benefits

  • Simplified configuration setup for developers
  • Better TypeScript integration
  • Modern, maintainable ESLint configuration structure
  • Consistent code style enforcement with improved defaults

Additional Updates

  • Updated READMEs across packages to reflect the new changes
  • Modified test suites to accommodate the new configuration structure
  • Maintained existing style preferences (semicolons, brace style) while adopting the new system

The ESLint configuration is migrated from @shopify/eslint-plugin to @antfu/eslint-config, 
providing better TypeScript support and adopting the new flat config format. Updates 
documentation and tests to reflect the new configuration approach.
@amazon-q-developer
Copy link
Author

Resolves #7

@amazon-q-developer
Copy link
Author

To provide feedback, I recommend leaving inline comments for best results. Navigate to the Files changed tab and leave comments on the proposed code changes. Choose Start review for each comment, and then choose Request changes, and I'll propose revised changes. If creating PR-level comments, include as much detail as possible in your feedback (for example, file name, class name, line number).

@amazon-q-developer
Copy link
Author

⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done

@amazon-q-developer
Copy link
Author

✅ I finished the code review, and didn't find any security or code quality issues.

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.

1 participant