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

Remove recursiveBlacklist option from jest-validate #10650

Merged
merged 4 commits into from
Dec 5, 2020
Merged

Remove recursiveBlacklist option from jest-validate #10650

merged 4 commits into from
Dec 5, 2020

Conversation

wojtekmaj
Copy link
Contributor

Summary

Part of #10235.

Follow-up after #10649.

Closes #10235.

Motivation

Part of continuous effort to get rid of non-inclusive terms like "whitelist" and "blacklist" implying that white = good and black = bad.

Test plan

Since this only removed deprecated alias for recursiveDenylist, the only change required was to remove the test that tested this very alias. recursiveDenylist had tests in place already, so the test plan is... To have them still working, I guess :)

@SimenB SimenB added this to the Jest 27 milestone Oct 19, 2020
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.

Thanks!

@SimenB SimenB changed the title [breaking] Remove recursiveBlacklist option from jest-validate Remove recursiveBlacklist option from jest-validate Dec 5, 2020
@SimenB SimenB merged commit 8a5c4d5 into jestjs:master Dec 5, 2020
@wojtekmaj wojtekmaj deleted the remove-recursiveBlacklist branch December 7, 2020 06:34
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace non-inclusive terms used in configuration of jest-validate
3 participants