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

Release v25 #905

Merged
merged 24 commits into from
Oct 10, 2021
Merged

Release v25 #905

merged 24 commits into from
Oct 10, 2021

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Sep 20, 2021

Just to keep track.

I believe for semantic-release to be happy we need to merge this (rather than squash or rebase), so I'll toggle the setting allowing it when we're ready to roll this.

Missing:

SimenB and others added 14 commits September 13, 2021 11:51
BREAKING CHANGE: Drop support for Node 10 and 15
# [25.0.0-next.1](v24.4.0...v25.0.0-next.1) (2021-09-13)

### Bug Fixes

* stop testing on Node 10 and 15 ([#891](#891)) ([bcd8d11](bcd8d11))

### BREAKING CHANGES

* Drop support for Node 10 and 15
BREAKING CHANGE: Drop support for ESLint 5
# [25.0.0-next.2](v25.0.0-next.1...v25.0.0-next.2) (2021-09-13)

### Bug Fixes

* stop testing ESLint 5 ([#893](#893)) ([47a0138](47a0138))

### BREAKING CHANGES

* Drop support for ESLint 5
BREAKING CHANGE: Removes rules `no-expect-resolves`, `no-truthy-falsy`, `no-try-expect`, and `prefer-inline-snapshots`
# [25.0.0-next.3](v25.0.0-next.2...v25.0.0-next.3) (2021-09-17)

### Features

* remove deprecated rules ([#661](#661)) ([e8f16ec](e8f16ec))

### BREAKING CHANGES

* Removes rules `no-expect-resolves`, `no-truthy-falsy`, `no-try-expect`, and `prefer-inline-snapshots`
# [25.0.0-next.4](v25.0.0-next.3...v25.0.0-next.4) (2021-09-20)

### Bug Fixes

* mark rules that suggest fixes with `hasSuggestion` for ESLint v8 ([#898](#898)) ([ec0a21b](ec0a21b))
* use correct property `hasSuggestions` rather than `hasSuggestion` ([#899](#899)) ([dfd2368](dfd2368))
@G-Rath
Copy link
Collaborator

G-Rath commented Sep 22, 2021

@SimenB are you wanting to land #905 as a breaking change, or are you wanting to get it landed before the new major so that folks can get that fix without needing to do the major upgrade?

I'm fine with either, but if it's the latter then I think we should apply the parseJestVersion code + #897 (which really should be done together) first as tbh that is a bug fix.

@SimenB
Copy link
Member Author

SimenB commented Sep 22, 2021

Current thinking is that #903 (which I assume you meant to link to) is a semver minor, so the plan is to merge this after #889 lands

@G-Rath
Copy link
Collaborator

G-Rath commented Sep 22, 2021

Actually I meant to link to #889 🤦

@SimenB
Copy link
Member Author

SimenB commented Sep 22, 2021

Ah. Plan is to land that as the last patch of 24 then move on with 25 🙂 Just missing the docs thing in #897, then I'll land all three (those two and this)

@G-Rath
Copy link
Collaborator

G-Rath commented Sep 25, 2021

@SimenB I just remembered about #864 😅

That rule opens up the possibility of deprecating prefer-to-be-undefined & prefer-to-be-null (which the PR doesn't do because they're in our style config), so if you're happy with that we could land it now and remove those two rules in this major, replacing them with the new rule in our style config.

The PR is all good to go if you're happy with that :)

@G-Rath
Copy link
Collaborator

G-Rath commented Sep 25, 2021

@SimenB have opened #910 which I think we should merge before the new major too.

@G-Rath
Copy link
Collaborator

G-Rath commented Sep 25, 2021

@SimenB also random thought: should we rename lowercase-name to something like prefer-lowercase-title? I'm not too fussed, but it struck me as I was looking over some issues that it's sort of an odd duck vs our other rules (e.g. no-identical-title, valid-title).

@G-Rath
Copy link
Collaborator

G-Rath commented Sep 25, 2021

Another naming thought: might make sense to rename valid-describe to valid-describe-callback, since that is actually what the rule is checking vs e.g. valid-title which applies to describes.

@SimenB
Copy link
Member Author

SimenB commented Sep 29, 2021

@SimenB I just remembered about #864 😅

That rule opens up the possibility of deprecating prefer-to-be-undefined & prefer-to-be-null (which the PR doesn't do because they're in our style config), so if you're happy with that we could land it now and remove those two rules in this major, replacing them with the new rule in our style config.

The PR is all good to go if you're happy with that :)

Let's do it 👍


@SimenB also random thought: should we rename lowercase-name to something like prefer-lowercase-title? I'm not too fussed, but it struck me as I was looking over some issues that it's sort of an odd duck vs our other rules (e.g. no-identical-title, valid-title).

Seems reasonable to me 🙂


Another naming thought: might make sense to rename valid-describe to valid-describe-callback, since that is actually what the rule is checking vs e.g. valid-title which applies to describes.

Yeah, agreed

SimenB and others added 7 commits September 29, 2021 09:54
# [25.0.0-next.5](v25.0.0-next.4...v25.0.0-next.5) (2021-09-29)

### Bug Fixes

* **no-deprecated-functions:** remove `process.cwd` from resolve paths ([#889](#889)) ([6940488](6940488))
* **no-identical-title:** always consider `.each` titles unique ([#910](#910)) ([a41a40e](a41a40e))
* **valid-expect-in-promise:** support `finally` ([#914](#914)) ([9c89855](9c89855))
* **valid-expect-in-promise:** support additional test functions ([#915](#915)) ([4798005](4798005))

### Features

* create `prefer-expect-resolves` rule ([#822](#822)) ([2556020](2556020))
* create `prefer-to-be` rule ([#864](#864)) ([3a64aea](3a64aea))
* **require-top-level-describe:** support enforcing max num of describes ([#912](#912)) ([14a2d13](14a2d13))
* **valid-title:** allow custom matcher messages ([#913](#913)) ([ffc9392](ffc9392))
# [25.0.0-next.6](v25.0.0-next.5...v25.0.0-next.6) (2021-10-10)

### Bug Fixes

* **lowercase-name:** consider skip and only prefixes for ignores ([#923](#923)) ([8716c24](8716c24))
* **prefer-to-be:** don't consider RegExp literals as `toBe`-able ([#922](#922)) ([99b6d42](99b6d42))

### Features

* create `require-hook` rule ([#929](#929)) ([6204b31](6204b31))
* deprecate `prefer-to-be-null` rule ([4db9161](4db9161))
* deprecate `prefer-to-be-undefined` rule ([fa08f09](fa08f09))
* **valid-expect-in-promise:** re-implement rule ([#916](#916)) ([7a49c58](7a49c58))
BREAKING CHANGE: recommend `prefer-to-be` rule
BREAKING CHANGE: removed `prefer-to-be-null` rule
BREAKING CHANGE: removed `prefer-to-be-undefined` rule
G-Rath and others added 3 commits October 10, 2021 11:22
BREAKING CHANGE: renamed `lowercase-name` to `prefer-lowercase-title`
BREAKING CHANGE: renamed `valid-describe` to `valid-describe-callback`
# [25.0.0-next.7](v25.0.0-next.6...v25.0.0-next.7) (2021-10-10)

### Features

* add `prefer-to-be` to style ruleset ([2a3376f](2a3376f))
* **lowercase-name:** rename to `prefer-lowercase-title` ([b860084](b860084))
* **prefer-to-be-null:** remove rule ([809bcda](809bcda))
* **prefer-to-be-undefined:** remove rule ([3434d9b](3434d9b))
* **valid-describe:** rename to `valid-describe-callback` ([f3e9e9a](f3e9e9a))

### BREAKING CHANGES

* **valid-describe:** renamed `valid-describe` to `valid-describe-callback`
* **lowercase-name:** renamed `lowercase-name` to `prefer-lowercase-title`
* **prefer-to-be-undefined:** removed `prefer-to-be-undefined` rule
* **prefer-to-be-null:** removed `prefer-to-be-null` rule
* recommend `prefer-to-be` rule
@SimenB SimenB marked this pull request as ready for review October 10, 2021 09:26
@SimenB
Copy link
Member Author

SimenB commented Oct 10, 2021

:shipit:

@SimenB SimenB merged commit 2dce752 into main Oct 10, 2021
@SimenB SimenB deleted the next branch October 10, 2021 09:26
@github-actions
Copy link

🎉 This PR is included in version 25.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants