Skip to content

v3.0.0 breaking changes #1998

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 29 commits into from
May 21, 2020
Merged

v3.0.0 breaking changes #1998

merged 29 commits into from
May 21, 2020

Conversation

bradzacher
Copy link
Member

@bradzacher bradzacher commented May 10, 2020

BREAKING CHANGE:

This is just a PR to increase visibility of breaking changes that are will be merged for the upcoming 3.0.0 release.

We will rebase merge this PR into master to preserve the history.

These changes are available on NPM under the rc-v3 tag

Release notes:
https://github.com/typescript-eslint/typescript-eslint/releases/tag/v3.0.0-alpha.23

@bradzacher bradzacher added the breaking change This change will require a new major version to be released label May 10, 2020
@codecov
Copy link

codecov bot commented May 10, 2020

Codecov Report

Merging #1998 into master will decrease coverage by 0.04%.
The diff coverage is 79.60%.

@@            Coverage Diff             @@
##           master    #1998      +/-   ##
==========================================
- Coverage   93.68%   93.63%   -0.05%     
==========================================
  Files         172      171       -1     
  Lines        7871     7717     -154     
  Branches     2263     2200      -63     
==========================================
- Hits         7374     7226     -148     
+ Misses        233      230       -3     
+ Partials      264      261       -3     
Flag Coverage Δ
#unittest 93.63% <79.60%> (-0.05%) ⬇️
Impacted Files Coverage Δ
...gin-internal/src/rules/no-poorly-typed-ts-props.ts 88.88% <ø> (ø)
packages/eslint-plugin/src/rules/ban-ts-comment.ts 100.00% <ø> (ø)
packages/eslint-plugin/src/rules/comma-spacing.ts 97.77% <ø> (ø)
...int-plugin/src/rules/consistent-type-assertions.ts 86.48% <ø> (ø)
...-plugin/src/rules/explicit-function-return-type.ts 100.00% <ø> (ø)
...plugin/src/rules/explicit-module-boundary-types.ts 82.60% <ø> (ø)
...kages/eslint-plugin/src/rules/func-call-spacing.ts 96.87% <ø> (ø)
...n/src/rules/indent-new-do-not-use/OffsetStorage.ts 100.00% <ø> (ø)
.../eslint-plugin/src/rules/member-delimiter-style.ts 94.44% <ø> (ø)
...kages/eslint-plugin/src/rules/naming-convention.ts 88.51% <ø> (ø)
... and 67 more

@typescript-eslint typescript-eslint deleted a comment from typescript-eslint bot May 11, 2020
@bradzacher bradzacher force-pushed the v3 branch 2 times, most recently from a8b8a6c to c1c6766 Compare May 14, 2020 08:28
@Bnaya
Copy link

Bnaya commented May 15, 2020

Need to update peer dependencies semver range:
Peer dependency warning:
warning " > @typescript-eslint/eslint-plugin@3.0.0-alpha.23" has incorrect peer dependency "@typescript-eslint/parser@^2.0.0".

Thanks :)

OliverSieweke and others added 19 commits May 21, 2020 09:27
@bradzacher bradzacher marked this pull request as ready for review May 21, 2020 16:32
SourceCode.isSpaceBetween only exists in 6.7.0, so isSpaceBetweenTokens is a valid alternative until older versions are dead.
…n] remove unsafe fixers

These fixers are very unsafe and even providing default options for them are a bad idea.
This ensures that they are suggestion fixers in all cases, and removes the options that allow them not to be.
This just creates a clearer entrypoint for contributors, and helps clearly define exactly what is exported from the modules.
@bradzacher bradzacher merged commit 3dfc46d into master May 21, 2020
@bradzacher bradzacher deleted the v3 branch May 21, 2020 18:53
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking change This change will require a new major version to be released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants