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

feat(no-large-snapshots): deprecate whitelistedSnapshots for new name #632

Merged
merged 1 commit into from Jul 30, 2020

Conversation

G-Rath
Copy link
Collaborator

@G-Rath G-Rath commented Jul 26, 2020

I'd be happy to deprecate passing in RegExp instances at the same time if we want to simplify things. I don't use the rule myself so not sure how heavily the RegExp support is used.

@G-Rath G-Rath requested a review from SimenB July 26, 2020 06:21
@G-Rath G-Rath force-pushed the rename-no-large-snapshots-property branch from 7b2ae7c to 7e1786a Compare July 28, 2020 04:28
@G-Rath G-Rath force-pushed the rename-no-large-snapshots-property branch from 7e1786a to 02eee8a Compare July 28, 2020 04:35
Copy link
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

Should we just rename in the next major? I think we should push through and publish a new major pretty soon

@G-Rath
Copy link
Collaborator Author

G-Rath commented Jul 30, 2020

Yes (thats the plan) and yes - I think we're pretty good to go for the new major - the main thing left is to decide on what rules to make recommended.

@G-Rath G-Rath merged commit 706f5c2 into master Jul 30, 2020
@G-Rath G-Rath deleted the rename-no-large-snapshots-property branch July 30, 2020 10:13
github-actions bot pushed a commit that referenced this pull request Jul 30, 2020
# [23.20.0](v23.19.0...v23.20.0) (2020-07-30)

### Features

* **no-large-snapshots:** deprecate `whitelistedSnapshots` for new name ([#632](#632)) ([706f5c2](706f5c2))
@github-actions
Copy link

🎉 This PR is included in version 23.20.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.

None yet

2 participants