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 linter packages (major) #244

Merged
merged 1 commit into from Mar 23, 2021

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Mar 13, 2021

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@commitlint/cli 11.0.0 -> 12.0.1 age adoption passing confidence
@commitlint/config-conventional 11.0.0 -> 12.0.1 age adoption passing confidence
eslint-config-prettier 7.2.0 -> 8.1.0 age adoption passing confidence

Release Notes

conventional-changelog/commitlint

v12.0.1

Compare Source

Bug Fixes

v12.0.0

Compare Source

Bug Fixes
Features
BREAKING CHANGES
  • resolve-extends: The order of the extends resolution is changed from right-to-left to left-to-right
prettier/eslint-config-prettier

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).


Renovate configuration

📅 Schedule: "on saturday" (UTC).

🚦 Automerge: Enabled.

♻️ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 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 added the renovate label Mar 13, 2021
@coveralls
Copy link

coveralls commented Mar 13, 2021

Pull Request Test Coverage Report for Build 1134

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 93.939%

Totals Coverage Status
Change from base Build 1126: 0.0%
Covered Lines: 47
Relevant Lines: 48

💛 - Coveralls

@renovate renovate bot force-pushed the renovate/major-linter-packages branch from 063bd81 to 998ae1c Compare March 23, 2021 12:39
@coston coston merged commit 04beafd into master Mar 23, 2021
@coston coston deleted the renovate/major-linter-packages branch March 23, 2021 12: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

3 participants