search

ehowey / gatsby-theme-catalyst

An opinionated set of integrated themes and starters as a boilerplate to accelerate development with GatsbyJS
MIT License
163 stars 15 forks source link

Excessive eslint warnings in development mode #1153

Closed ehowey closed 3 years ago

ehowey commented 3 years ago

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: https://github.com/yannickcr/eslint-plugin-react/pull/2917

ehowey commented 3 years ago

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