Skip to content

chore: upgrade ESLint v6 and related dependencies #3698

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

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

kaicataldo
Copy link


name: "\U0001F41B Bug fix or new feature"
about: Fixing a problem with Redux

PR Type

Upgrading dependencies. Apologies if this is the wrong template; it didn't seem like any of them really fit.

Does this PR add a new feature, or fix a bug?

Neither! This upgrades ESLint and the dependencies related to it.

Why should this PR be included?

I'm a maintainer of ESLint and have been working on updating our regression build to ESLint v6. This repo is one we test in our build, and I noticed that this repo is still using ESLint v5. We're currently working on v7, and upgrading it now will help make the next major update smoother :)

Checklist

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Is there an existing issue for this PR?
    • link issue here
  • Have the files been linted and formatted?
  • Have the docs been updated to match the changes in the PR?
  • Have the tests been updated to match the changes in the PR?
  • Have you run the tests locally to confirm they pass?

@@ -21,6 +21,7 @@ module.exports = {
rules: {
'jsx-a11y/href-no-hash': 'off',
'no-unused-vars': 'off',
'@typescript-eslint/no-unused-expressions': 'off',
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like this rule is turned on in one of the updated configs, and there are a good number of warnings. Turning off since it's not a style this codebase enforces.

@netlify
Copy link

netlify bot commented Feb 12, 2020

Deploy preview for redux-docs ready!

Built with commit 49de38d

https://deploy-preview-3698--redux-docs.netlify.com

@kaicataldo kaicataldo changed the title Upgrade: ESLint v6 chore: upgrade ESLint v6 and related dependencies Feb 12, 2020
@timdorr
Copy link
Member

timdorr commented Feb 12, 2020

Thanks!

@timdorr timdorr merged commit 9819b5e into reduxjs:master Feb 12, 2020
@kaicataldo kaicataldo deleted the upgrade-eslint branch February 12, 2020 04:27
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
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.

2 participants