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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update eslint-plugin-react to the latest version 馃殌 #191

Closed
wants to merge 1 commit into from

Conversation

greenkeeper[bot]
Copy link

@greenkeeper greenkeeper bot commented Aug 21, 2017

Version 7.3.0 of eslint-plugin-react just got published.

Dependency eslint-plugin-react
Current Version 4.3.0
Type devDependency

The version 7.3.0 is not covered by your current version range.

Without accepting this pull request your project will work just like it did before. There might be a bunch of new features, fixes and perf improvements that the maintainers worked on for you though.

I recommend you look into these changes and try to get onto the latest version of eslint-plugin-react.
Given that you have a decent test suite, a passing build is a strong indicator that you can take advantage of these changes by merging the proposed change into your project. Otherwise this branch is a great starting point for you to work on the update.


Release Notes v7.3.0

Added

  • Add checks for propTypes, contextTypes and childContextTypes to no-typos (#213 @DianaSuvorova)

Fixed

  • Fix boolean-prop-naming crash (#1369 @EvHaus)
  • Fix no-typos crash (#1353 @jseminck)
  • Fix require-default-props stopping when it finds a component without props (#1380 @brgibson)
  • Fix no-direct-mutation-state detection with nested components (#1382)

Changed

Commits

The new version differs by 749 commits.

  • 162b92b Update CHANGELOG and bump version
  • 686ab89 Merge pull request #1375 from jseminck/more-robust-no-typos-fix
  • 9805f70 Fix no-direct-mutation-state detection with nested components (fixes #1382)
  • 6af4bca Merge pull request #1383 from mjomble/patch-1
  • a05bc0a Fixed links in changelog
  • 48e8dc1 Merge pull request #1380 from brgibson/master
  • f371368 require-default-props - fixing an edge case where linting stops when it finds a component without props - should move on to the next component in the file
  • 5e25a5e Re-add FIXME and disable the warning through eslint-disable-next-line
  • d00e9e6 Remove FIXME as it fails linting
  • 0cf2415 Add comment to the eslint parent definition issue
  • 02ed7d0 Merge pull request #1371 from DianaSuvorova/i213
  • 1292807 Add comment explaining try/catch
  • fa890cb Tests and explaining the case that is currently not supported.
  • 1301fe9 More robust no-typos fix
  • 59a3f6a [no-typo] rule enhancement checking for a correct prop type spelling

There are 250 commits in total.

See the full diff

Not sure how things should work exactly?

There is a collection of frequently asked questions and of course you may always ask my humans.


Your Greenkeeper Bot 馃尨

@coveralls
Copy link

coveralls commented Aug 21, 2017

Coverage Status

Coverage remained the same at 90.672% when pulling 9d6d1cf on greenkeeper/eslint-plugin-react-7.3.0 into a50657b on master.

@kommander kommander closed this Aug 24, 2017
@kommander kommander deleted the greenkeeper/eslint-plugin-react-7.3.0 branch October 21, 2017 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants