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

chore(deps): update eslint (major) #230

Merged
merged 2 commits into from
Apr 19, 2021
Merged

chore(deps): update eslint (major) #230

merged 2 commits into from
Apr 19, 2021

Conversation

renovate[bot]
Copy link

@renovate renovate bot commented Dec 1, 2020

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
eslint-config-prettier ^6.11.0 -> ^8.0.0 age adoption passing confidence
eslint-plugin-ember ^9.0.0 -> ^10.0.0 age adoption passing confidence

Release Notes

prettier/eslint-config-prettier

v8.2.0

Compare Source

  • Added: [flowtype/quotes]. Thanks to Martin Zlámal (@​mrtnzlml)!

v8.1.0

Compare Source

  • Added: [flowtype/object-type-curly-spacing].
  • Added: Dummy files for the configs removed in 8.0.0. The dummy files throw an error that try to guide you how to upgrade.

v8.0.0

Compare Source

  • Changed: All configs have been merged into one!

    To upgrade, change:

    {
      "extends": [
        "some-other-config-you-use",
        "prettier",
        "prettier/@​typescript-eslint",
        "prettier/babel",
        "prettier/flowtype",
        "prettier/react",
        "prettier/standard",
        "prettier/unicorn",
        "prettier/vue"
      ]
    }

    Into:

    {
      "extends": [
        "some-other-config-you-use",
        "prettier"
      ]
    }

    The "prettier" config now includes not just ESLint core rules, but also rules from all plugins. Much simpler!

    So … what’s the catch? Why haven’t we done this earlier? Turns out it’s just a sad mistake. I (@​lydell) was confused when testing, and thought that turning off unknown rules in a config was an error. Thanks to Georgii Dolzhykov (@​thorn0) for pointing this out!

    If you use [eslint-plugin-prettier], all you need is [plugin:prettier/recommended]:

    {
      "extends": [
        "some-other-config-you-use",
        "plugin:prettier/recommended"
      ]
    }

    (The ["prettier/prettier" config][prettier-prettier-config] still exists separately. It’s the odd one out. The main "prettier" config does not include the rules from it.)

  • Changed: The CLI helper tool now only prints warnings for [arrow-body-style] and [prefer-arrow-callback], just like other “special rules.” This means that if you’ve decided to use those rules and [eslint-plugin-prettier] at the same time, you’ll get warnings but exit code zero (success).

v7.2.0

Compare Source

  • Added: [@​typescript-eslint/object-curly-spacing].
  • Added: [react/jsx-newline].

v7.1.0

Compare Source

  • Added: [unicorn/empty-brace-spaces]. Thanks to fisker Cheung (@​fisker)!

v7.0.0

Compare Source

  • Changed: At least ESLint 7.0.0 is now required.

  • Changed: [arrow-body-style] and [prefer-arrow-callback] are no longer turned off by default. They only need to be turned off if you use [eslint-plugin-prettier]. If you do, add "prettier/prettier" to your "extends" array to turn them off again.

    {
      "extends": ["prettier", "prettier/prettier"],
      "plugins": ["prettier"],
      "rules": {
        "prettier/prettier": "error"
      }
    }

    Alternatively, update [eslint-plugin-prettier] to version 3.2.0 or later which automatically turns off these two rules in its "plugin:prettier/recommended" config.

    The CLI helper tool only warns about these rules if you have the "prettier/prettier" rule enabled for a file.

  • Changed: no-tabs is now a validatable rule. If you use it, you should enable allowIndentationTabs so that the rule works regardless of your Prettier config:

    {
      "rules": {
        "no-tabs": ["error", { "allowIndentationTabs": true }]
      }
    }
  • Changed: The CLI helper tool is now called just eslint-config-prettier instead of eslint-config-prettier-check. This is so that npx eslint-config-prettier always works regardless of whether you have already installed eslint-config-prettier or not: If you have, the local installation is used; if you haven’t, npx downloads a temporary copy.

  • Changed: The CLI helper tool no longer requires you to pipe the output of eslint --print-config to it. Instead, it does that automatically for you via ESLint API:s added in ESLint v7.

    Before:

    npx eslint --print-config index.js | npx eslint-config-prettier-check
    

    After:

    npx eslint-config-prettier index.js
    
  • Improved: The npm package is now 75% smaller.

v6.15.0

Compare Source

  • Added: [@​typescript-eslint/space-infix-ops]. Thanks to Masafumi Koba (@​ybiquitous)!

v6.14.0

Compare Source

  • Added: New [eslint-plugin-vue] rules: [vue/array-bracket-newline] and [vue/block-tag-newline]. Thanks to @​xcatliu!

v6.13.0

Compare Source

  • Added: New rules in [eslint-plugin-vue] 7.0 (which supports Vue 3.0). Thanks to @​xcatliu!

v6.12.0

Compare Source

  • Added: [@​typescript-eslint/comma-dangle]. Thanks to Masafumi Koba (@​ybiquitous)!
ember-cli/eslint-plugin-ember

v10.3.0

Compare Source

🚀 Enhancement
🐛 Bug Fix
  • #​1115 Fix crash with dynamic/variable route name in no-shadow-route-definition rule (@​bmish)
  • #​1102 Fix crash with this.extend() in no-classic-classes rule (@​bmish)
  • #​1114 Ensure rules validate arrays in options to have at least one item and unique items (@​bmish)
  • #​1103 Only calculate source module name once in import util function for slight optimization (@​bmish)
  • #​1081 Update avoid-leaking-state-in-ember-objects rule to apply to mixins (@​jaydgruber)
Committers: 3

v10.2.0

Compare Source

🚀 Enhancement
🐛 Bug Fix
Committers: 2

v10.1.2

Compare Source

🐛 Bug Fix
🏠 Internal
Committers: 2

v10.1.1

Compare Source

🐛 Bug Fix
  • #​1059 Do not warn about Glimmer lifecycle hooks on classic components in no-component-lifecycle-hooks rule (@​Turbo87)
🏠 Internal
Committers: 2

v10.1.0

Compare Source

🚀 Enhancement
🐛 Bug Fix
🏠 Internal
Committers: 4

v10.0.2

Compare Source

🐛 Bug Fix
📝 Documentation
  • #​1028 Mention Ember 3.13 minimum version to use tracked properties and Glimmer components in some rule docs (@​bmish)
Committers: 1

v10.0.1

Compare Source

🐛 Bug Fix
Committers: 1

v10.0.0

Compare Source

💥 Breaking Change
📝 Documentation
Committers: 3

v9.6.0

Compare Source

🚀 Enhancement
🐛 Bug Fix
Committers: 1

v9.5.0

Compare Source

🚀 Enhancement
🐛 Bug Fix
  • #​998 Fix autofix for array element access at beginning of path string in no-get rule (@​bmish)
Committers: 2

v9.4.0

Compare Source

🚀 Enhancement
🏠 Internal
  • #​991 Add test to ensure rule test files have correct test suite name (@​bmish)
Committers: 4

v9.3.0

Compare Source

🚀 Enhancement
🐛 Bug Fix
Committers: 2

v9.2.0

Compare Source

🚀 Enhancement
🐛 Bug Fix
Committers: 2

v9.1.1

Compare Source

🐛 Bug Fix
Committers: 1

v9.1.0

Compare Source

🚀 Enhancement
📝 Documentation
🏠 Internal
Committers: 1

Configuration

📅 Schedule: "after 10pm every weekday,before 5am every weekday,every weekend" (UTC).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box.

This PR has been generated by WhiteSource Renovate. View repository job log here.

@renovate renovate bot changed the title chore(deps): update dependency eslint-plugin-ember to v10 chore(deps): update eslint (major) Dec 5, 2020
@renovate renovate bot changed the title chore(deps): update eslint (major) Update eslint (major) Feb 9, 2021
@drewlee
Copy link
Contributor

drewlee commented Feb 25, 2021

This one is going to need some work. It introduces a bunch of linting errors that need to be addressed.

@renovate renovate bot changed the title Update eslint (major) chore(deps): update eslint (major) Mar 2, 2021
@drewlee drewlee merged commit 0f4e5cf into master Apr 19, 2021
@renovate renovate bot deleted the renovate/major-eslint branch April 19, 2021 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants