Skip to content

Comments

ESLint Configuration Fixes (For Review)#10

Open
cameronrye wants to merge 2 commits intomainfrom
eslint-fixes-review
Open

ESLint Configuration Fixes (For Review)#10
cameronrye wants to merge 2 commits intomainfrom
eslint-fixes-review

Conversation

@cameronrye
Copy link
Owner

@cameronrye cameronrye commented Nov 12, 2025

This PR reintroduces the ESLint configuration fixes that were previously merged in PR #7 without review.

Changes

This PR includes two commits:

  1. Fix ESLint configuration for Playwright files (975563b)

    • Adds ESLint configuration for Playwright test files
    • Includes development tooling setup (.editorconfig, .prettierrc, etc.)
    • Adds Storybook configuration and stories
    • Adds Playwright test configuration and test files
  2. Fix ESLint configuration for browser globals (992426e)

    • Fixes ESLint configuration to properly recognize browser globals (window, document, etc.)
    • Adds comprehensive example site components
    • Adds data files for examples, tutorials, and documentation
    • Removes .eslintignore in favor of inline configuration

Context

  • Previous PR: ESLint Configuration Fixes #7 (merged on 2025-11-12 without review)
  • Reverted: The changes were reverted in commit 4c27bd2 to allow for proper review
  • Files Changed: 103 files, 24,008 additions, 2,105 deletions

Review Notes

Please review the following areas carefully:

  • ESLint configuration changes in eslint.config.mjs
  • Browser globals configuration for example site TypeScript files
  • Playwright configuration and test setup
  • Development tooling configuration files
  • New example site components and data files

This PR should be reviewed before merging to ensure all configuration changes are appropriate and complete.

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