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

fix(deps): update dependency @typescript-eslint/eslint-plugin to v6 #2844

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jul 23, 2023

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@typescript-eslint/eslint-plugin 5.62.0 -> 6.2.1 age adoption passing confidence

Release Notes

typescript-eslint/typescript-eslint (@​typescript-eslint/eslint-plugin)

v6.2.1

Compare Source

Bug Fixes
  • eslint-plugin: [no-inferrable-types] apply also for parameter properties (#​7288) (67f93b1)
  • scope-manager: correct decorators(.length) check in ClassVisitor for methods (#​7334) (abbb6c2)

You can read about our versioning strategy and releases on our website.

v6.2.0

Compare Source

Bug Fixes
  • eslint-plugin: [member-ordering] account for repeated names (#​6864) (d207b59)
  • eslint-plugin: [no-unsafe-enum-comparison] exempt bit shift operators (#​7074) (b3e0e75)
  • eslint-plugin: [prefer-nullish-coalescing] handle case when type of left side is null or undefined (#​7225) (b62affe)
  • eslint-plugin: use a default export for the rules type (#​7266) (af77a1d)
Features
  • eslint-plugin: [class-methods-use-this] add extension rule (#​6457) (18ea3b1)
  • eslint-plugin: sync getFunctionHeadLoc implementation with upstream (#​7260) (f813147)

You can read about our versioning strategy and releases on our website.

v6.1.0

Compare Source

Bug Fixes
  • eslint-plugin: [comma-dangle] fixed crash from undefined predicate.ignore (#​7223) (d368164)
  • eslint-plugin: [no-floating-promises] false negative calling .then with second argument undefined (#​6881) (606a52c), closes #​6850
  • eslint-plugin: [no-floating-promises] finally should be transparent to unhandled promises (#​7092) (2a4421c)
  • eslint-plugin: [no-unnecessary-type-constraint] correctly fix in cts/mts files (#​6795) (1404796)
  • eslint-plugin: [no-unused-vars] check if any variable definition is exported (#​6873) (587ac30), closes #​6188
  • eslint-plugin: fix schemas across several rules and add schema tests (#​6947) (dd31bed)
  • eslint-plugin: include the rules types in the package (#​7215) (a3da11d)
Features
  • typescript-estree: add EXPERIMENTAL_useProjectService option to use TypeScript project service (#​6754) (6d3d162)

You can read about our versioning strategy and releases on our website.

v6.0.0

Compare Source

Bug Fixes
  • bumped ts-api-utils to 0.0.39 (#​6497) (217c710)
  • correct handling of additionalProperties in object schemas (#​6939) (489c7a5)
  • correct lint error after merge (277fdb5)
  • eslint-plugin: [explicit-module-boundary-types] remove shouldTrackReferences option from schema (#​5399) (6d32734)
  • eslint-plugin: allow parser@^6.0.0 (#​6630) (92908bd)
  • eslint-plugin: remove valid-typeof disable in eslint-recommended (#​5381) (b82df5e)
  • fix illegal decorator check (#​6723) (c456f8c)
  • rename typeParameters to typeArguments where needed (#​5384) (08d757b)
  • replace tsutils with ts-api-tools (#​6428) (79327b4)
  • type-utils: treat intrinsic types as if they are from lib and never match error types (#​6869) (ecb57de)
  • typescript-estree: account for namespace nesting in AST conversion (#​6272) (09e3877)
  • update exports field in package.json files (#​6550) (53776c2)
chore
Features
BREAKING CHANGES
  • eslint-plugin: Adds an additional class of checks to the rule
  • drop support for ESLint v6
  • drops support for node v17
  • Bumps the minimum supported range and removes handling for old versions
  • drops support for node v12

You can read about our versioning strategy and releases on our website.


Configuration

📅 Schedule: Branch creation - "after 10pm every weekday,before 5am every weekday,every weekend" (UTC), Automerge - At any time (no schedule defined).

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

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

🔕 Ignore: Close this PR and you won't be reminded about this update again.


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

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

@renovate renovate bot requested a review from a team as a code owner July 23, 2023 03:08
@renovate renovate bot added the needs approval/merge Major dependency updates, automatically added to renovate PRs. These need human approvals & merging label Jul 23, 2023
@vercel
Copy link

vercel bot commented Jul 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hedvig-dot-com ✅ Ready (Inspect) Visit Preview Aug 8, 2023 2:25pm

@robinandeer
Copy link
Contributor

I think we need to wait until the next eslint-config-next version is released

track: vercel/next.js#52848

@guilhermespopolin
Copy link
Contributor

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@renovate
Copy link
Contributor Author

renovate bot commented Aug 8, 2023

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@@ -5,7 +5,8 @@
"main": "eslint-config-custom.js",
"license": "MIT",
"dependencies": {
"@typescript-eslint/eslint-plugin": "5.62.0",
"@typescript-eslint/eslint-plugin": "6.2.1",
"@typescript-eslint/parser": "6.2.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

The was causing the issue: we needed to update the parser.

Comment on lines +7 to +8
'plugin:@typescript-eslint/recommended-type-checked',
'plugin:@typescript-eslint/stylistic-type-checked',
Copy link
Contributor

Choose a reason for hiding this comment

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

We're talking about a breaking change update here.
As it can be looked in their docs they changed the way typescript-eslint is configured.

Comment on lines +56 to +67
'@typescript-eslint/array-type': 'off',
'@typescript-eslint/consistent-type-definitions': 'off',
'@typescript-eslint/prefer-nullish-coalescing': 'off',
'@typescript-eslint/non-nullable-type-assertion-style': 'off',
'@typescript-eslint/no-redundant-type-constituents': 'off',
Copy link
Contributor

Choose a reason for hiding this comment

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

We're also talking about some new rules. I'm trying to disable them here just to move forward with it but we might want want to consider on enabling them and fix any issues in the future.
Now, I'm saying trying because there's some weird behaviour. Even though I'm disabling some rules in this file, the eslint rules to are still breaking. That's happening for old and new rules.

E.g: We're disabling @typescript-eslint/no-explicit-any in this file which is extended by ui, onboarding and store eslint configurations. However, eslint still complains about it.
The same happens for the new rules as well. Any hints on why is that?
@alebedev @robinandeer

@@ -76,15 +89,5 @@ module.exports = {
'@typescript-eslint/no-var-requires': 'off',
},
},
{
Copy link
Contributor

Choose a reason for hiding this comment

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

@alebedev Let me know if that makes sense to be removed or if there something that I'm not foreseeing.

Copy link
Contributor

Choose a reason for hiding this comment

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

It was necessary evil, glad it works without it now

Copy link
Contributor

@alebedev alebedev left a comment

Choose a reason for hiding this comment

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

Great job, let's merge!

@guilhermespopolin guilhermespopolin force-pushed the renovate/major-typescript-eslint-monorepo branch from fe4aae9 to aeaf187 Compare August 8, 2023 14:21
@guilhermespopolin guilhermespopolin merged commit f163ef5 into main Aug 8, 2023
4 checks passed
@guilhermespopolin guilhermespopolin deleted the renovate/major-typescript-eslint-monorepo branch August 8, 2023 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs approval/merge Major dependency updates, automatically added to renovate PRs. These need human approvals & merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants