Skip to content
This repository has been archived by the owner on Jun 17, 2022. It is now read-only.

Excessive eslint warnings in development mode #1153

Closed
ehowey opened this issue Feb 17, 2021 · 1 comment
Closed

Excessive eslint warnings in development mode #1153

ehowey opened this issue Feb 17, 2021 · 1 comment

Comments

@ehowey
Copy link
Owner

ehowey commented Feb 17, 2021

Describe the bug
Gatsby updated their base eslint config and removed a customization that I contributed to there over a year ago to deal with this.

You are likely seeing stuff like this:

warn ESLintError: 
/Users/erichowey/coding/gatsby-theme-catalyst/themes/gatsby-theme-catalyst-blog/src/components/templates/post-list-template.js
  12:31  warning  Imported JSX component h1 must be in PascalCase or SCREAMING_SNAKE_CASE  react/jsx-pascal-case
  18:15  warning  Imported JSX component h2 must be in PascalCase or SCREAMING_SNAKE_CASE  react/jsx-pascal-case

Expected behavior
No warnings for Theme-UI

Fix
An upstream fix is in the works but now involves actually updating eslint itself which is a bigger undertaking (and not one I am doing). You can follow progress here: jsx-eslint/eslint-plugin-react#2917

@ehowey
Copy link
Owner Author

ehowey commented Mar 29, 2021

This has been fixed, waiting on a new Gatsby release to include the updated config.

@ehowey ehowey closed this as completed Mar 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant