Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: remove no-if rule #1528

Merged
merged 1 commit into from
Mar 21, 2024
Merged

feat: remove no-if rule #1528

merged 1 commit into from
Mar 21, 2024

Conversation

G-Rath
Copy link
Collaborator

@G-Rath G-Rath commented Mar 21, 2024

BREAKING CHANGE

@SimenB
Copy link
Member

SimenB commented Mar 21, 2024

(not against it, but why? missing motivation/issue I think 😅 )

@G-Rath
Copy link
Collaborator Author

G-Rath commented Mar 21, 2024

Do you mean for the changelog or that you've forgotten the history? (I don't blame you if so😅)

We replaced it with no-conditional-in-test which does the same but with more conditions, but we kept forgetting to actually remove this rule in our follow-up majors...

#1049 & #1027

@SimenB
Copy link
Member

SimenB commented Mar 21, 2024

haha, both! 😅 The BREAKING CHANGE part of the commit message should include motivation/migration path.

@G-Rath
Copy link
Collaborator Author

G-Rath commented Mar 21, 2024

🤦 it's been so long I forgot that

BREAKING CHANGE: removed `no-if` in favor of `no-conditional-in-test`
@G-Rath G-Rath merged commit f976fc8 into next Mar 21, 2024
42 checks passed
@G-Rath G-Rath deleted the remove-no-if branch March 21, 2024 19:28
github-actions bot pushed a commit that referenced this pull request Mar 21, 2024
# [28.0.0-next.1](v27.9.0...v28.0.0-next.1) (2024-03-21)

### Features

* remove `no-if` rule ([#1528](#1528)) ([f976fc8](f976fc8))

### BREAKING CHANGES

* removed `no-if` in favor of `no-conditional-in-test`
Copy link

🎉 This PR is included in version 28.0.0-next.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Apr 6, 2024
# [28.0.0](v27.9.0...v28.0.0) (2024-04-06)

### Bug Fixes

* allow ESLint 9 as peer dependency ([#1547](#1547)) ([3c5e167](3c5e167))
* drop support for Node 19 ([#1548](#1548)) ([c87e388](c87e388))
* **no-large-snapshots:** avoid `instanceof RegExp` check for ESLint v9 compatibility ([#1542](#1542)) ([af4a9c9](af4a9c9))

### Features

* drop support for `@typescript-eslint/eslint-plugin` v5 ([#1530](#1530)) ([150e355](150e355))
* drop support for Node v14 ([#1527](#1527)) ([df5e580](df5e580))
* remove `no-if` rule ([#1528](#1528)) ([f976fc8](f976fc8))
* remove `snapshot` processor and `flat/snapshot` config ([#1532](#1532)) ([98087f9](98087f9))
* upgrade `@typescript-eslint/utils` to v6 ([#1508](#1508)) ([dc6e8cd](dc6e8cd))

### BREAKING CHANGES

* Node v19 is no longer supported
* removed unneeded `snapshot` processor and `flat/snapshot` config
* dropped support for `@typescript-eslint/eslint-plugin` v5
* dropped support for Node v14
* removed `no-if` in favor of `no-conditional-in-test`
Copy link

github-actions bot commented Apr 6, 2024

🎉 This PR is included in version 28.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants