Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

An in-range update of eslint-plugin-react is breaking the build 🚨 #71

Open
greenkeeper bot opened this issue Oct 1, 2019 · 9 comments
Open

Comments

@greenkeeper
Copy link
Contributor

greenkeeper bot commented Oct 1, 2019

The devDependency eslint-plugin-react was updated from 7.14.3 to 7.15.0.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

eslint-plugin-react is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Status Details
  • ci/circleci: Your tests failed on CircleCI (Details).

Commits

The new version differs by 30 commits.

  • b45219a Update CHANGELOG and bump version
  • a01198f [Dev Deps] update eslint-config-airbnb-base, eslint-plugin-import, sinon, typescript
  • ab77870 [Deps] update jsx-ast-utils, resolve
  • 32e27b7 [Fix] jsx-indent: Fix false positive when a jsx element is the last statement within a do expression (with tests)
  • 628a4a0 [fix] jsx-curly-brace-presence: fix jsx tags in braces
  • 3124ce0 [Docs] no-access-state-in-setstate: update grammar
  • 4a05fa2 [fix] display-name: Fix false positives
  • c6521ad Change istanbul output directory to ./coverage (default)
  • 5970651 [new] jsx-handler-name: allow false to disable eventHandlerPrefix/eventHandlerPropPrefix
  • 489ced3 [refactor] jsx-curly-braces-presence, jsx-one-expression-per-line, no-danger-with-children: add isWhiteSpaces to lib/util/jsx
  • 9b2db61 [fix] jsx-curly-brace-presence: report unnecessary curly braces with children on next line
  • 781bfff [new] sort-comp: add static-variables grouping
  • 7ccff10 [New] add jsx-no-useless-fragment rule
  • 66725bc [Fix] no-unused-prop-types: false positive with callback
  • 3c72a49 [eslint] disable some options and manually fix valid-jsdoc

There are 30 commits in total.

See the full diff

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.


Your Greenkeeper Bot 🌴

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Oct 1, 2019

After pinning to 7.14.3 your tests are passing again. Downgrade this dependency 📌.

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Oct 1, 2019

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 4 commits.

  • e336aef Update CHANGELOG and bump version
  • 04fbe74 [Fix] jsx-curly-brace-presence: bail out checks when JSXElements are passed as props
  • 20cc016 [Docs] prefer-es6-class: Fix typos
  • feba507 [Fix] jsx-curly-brace-presence: bail out on JSX inside props, for now

See the full diff

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Oct 4, 2019

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 6 commits.

  • fb3210b Update CHANGELOG and bump version
  • 6011505 [new] jsx-sort-default-props: make rule fixable
  • 6f14e16 [fix] jsx-no-useless-fragment: use array-includes over .includes for back compat
  • a1dee7c [fix] jsx-curly-brace-presence: allow necessary white-space literal
  • d7d2a01 [Fix] jsx-curly-brace-presence: warns incorrectly on trailing whitespace
  • c153a86 [Fix] no-unused-prop-types: false positive when nested destructuring

See the full diff

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Nov 29, 2019

Your tests are passing again with this update. Explicitly upgrade to this version 🚀

Release Notes for v7.17.0

Added

Fixed

Changed

Commits

The new version differs by 26 commits.

  • 027ebd9 Update CHANGELOG and bump version
  • 89911de [Deps] update jsx-ast-utils, object.fromentries, resolve
  • cfef664 [Dev Deps] update @types/eslint, @types/estree, @types/node, coveralls, typescript
  • d3b5d71 [New] jsx-no-target-blank: add allowReferrer option
  • 83ac4c0 [eslint] fix func-names and change object-shorthand to 'always'
  • 225b433 [Docs] jsx-first-prop-new-line: Fix documentation formatting
  • 62df1dd [Docs] prop-types: Update 'skipUndeclared' in rule options
  • 87d4cab [Docs] jsx-first-prop-new-line: fix wrong rule name
  • 55b605f [Fix] sort-prop-types, jsx-sort-default-props: disable broken autofix
  • 71c7d01 [Tests] temporarily pin eslint 6 tests to v6.6
  • 8093565 [New] jsx-handler-names: add checkLocalVariables option
  • 5be539b [Fix] no-typos: improve report location
  • d9d2193 [eslint] enable eslint-plugin-eslint-plugin
  • 39e4396 [Docs] jsx-props-no-multi-spaces: suggest using core rule instead
  • 9e4ad21 [Docs] jsx-first-prop-new-line: add rule options

There are 26 commits in total.

See the full diff

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Feb 1, 2020

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 58 commits.

  • 000d6b8 Update CHANGELOG and bump version
  • ffdf69a [Fix] jsx-indent: Does not check indents for JSXText
  • 182b045 [Docs] use markdown-magic to automatically sort all rules alphabetically
  • 45b9d32 [Docs] jsx-props-no-spreading: fix typo to use correct rule
  • f9aee94 [Fix] jsx-props-no-spreading: add support for namespaced jsx components
  • e69b113 [Fix] jsx-no-target-blank: allow rel to be an expression
  • fc6e406 [meta] fix changelog date
  • 50df78f [Fix] sort-comp: | isn’t a valid regex flag; u and s are
  • a7f6a8b [Tests] on node v13
  • 6a45142 Update CHANGELOG and bump version
  • d9e4941 [Deps] update array-includes, object.entries, object.fromentries, object.values, resolve
  • d1dc277 [Fix] no-unused-prop-types, no-unused-state: fix false positives when using TS type assertions
  • 99cea84 [New] function-component-definition: Enforce a specific function type for function components
  • c234d0f [Docs] no-render-return-value: Fix title
  • 5fac02c [Fix] jsx-pascal-case: false negative with namespacing

There are 58 commits in total.

See the full diff

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Feb 2, 2020

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 2 commits.

  • 37c4ef3 Update CHANGELOG and bump version
  • 3da6eb3 [Fix] jsx-indent: avoid a crash on non-string literals

See the full diff

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Feb 3, 2020

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 2 commits.

  • 0a717a5 Update CHANGELOG and bump version
  • 8b576be [Fix] jsx-indent: don't check literals not within JSX

See the full diff

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Mar 6, 2020

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 14 commits.

  • e2ed536 Update CHANGELOG and bump version
  • 5959b5f [fix] prefer-stateless-function: avoid crash on ts empty constructor
  • e231f44 [tests] no-unescaped-entities: skip test cases that are now parsing errors in [email protected]
  • 434b45f [Dev Deps] update @types/eslint, @types/estree, @types/node, eslint-plugin-eslint-plugin, eslint-plugin-import, typescript
  • a917dda [Deps] update resolve, xregexp
  • 09608cc [Fix] no-adjacent-inline-elements: avoid a crash
  • b833535 [Fix] no-unused-prop-types: Change the reporting to point to a more accurate node
  • edc5fe2 [New] style-prop-object: Add allow option
  • 9b5d6aa [Fix] self-closing-comp: consider JSXMemberExpression as component too
  • 598277e [New] jsx-pascal-case: Support unicode characters
  • 43e466c [readme] Small visual inconsistency
  • 03edb68 [docs] add react/ prefix to rule name, for consistency
  • b517b9e [Fix] no-unused-prop-types: make markPropTypesAsUsed work with TSEmptyBodyFunctionExpression AST node
  • 66c0d66 [Fix] displayName (but really, propTypes detection): do not crash on empty flow type spreads

See the full diff

@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented May 13, 2020

Your tests are still failing with this version. Compare changes

Release Notes for v7.20.0

Added

Fixed

Docs

Changed

  • [Deps] Move "semver" to devDependencies (#2595 @rajivshah3)
  • [eslint] remove operator-linebreak override (#2578 @golopot)
  • [Tests] button-has-type: ensure no mistakenly allowed identifiers named button/submit/reset (#2625 @golopot)
  • [Tests] displayName: add a test case (#2593 @golopot)
  • [Dev Deps] update @types/eslint, @types/estree, @types/node, @typescript-eslint/parser, coveralls, eslint-config-airbnb-base, eslint-plugin-import, typescript
Commits

The new version differs by 27 commits.

  • 25bf6cc Update CHANGELOG and bump version
  • 2598b1e [Dev Deps] update @types/eslint, @types/estree, @types/node, @typescript-eslint/parser, coveralls, eslint-config-airbnb-base, eslint-plugin-import, typescript
  • f94d851 [Fix] jsx-sort-props: only use localeCompare when case is ignored
  • c481a26 [Docs] Clean up examples in rule docs
  • 88d404d [eslint] remove operator-linebreak override
  • 304590a Merge pull request #2635 from ljharb/eslint7
  • 35030a0 [Tests] fix mistaken error property and add output property
  • 906b474 [New] support eslint v7
  • d66f467 [Fix] jsx-key: add a failing test case for optional chaining
  • da7a045 [Fix] no-unused-state: handle optional chaining
  • 9861469 [New] forbid-component-props/forbid-dom-props: Allow a custom message with forbid props
  • ab28224 [Fix] jsx-pascal-case: Do not consider namespaces when checking for DOM
  • 25b1936 [Fix] jsx-curly-spacing, jsx-no-bind, usedPropTypes util: avoid node.start and node.end
  • db4e471 [readme] Add Rules of Hooks to Other useful plugins section
  • b9d2eb5 [Fix] jsx-no-target-blank: allow no-referrer without noopener by default

There are 27 commits in total.

See the full diff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

0 participants