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

Replacing non-inclusive term blacklist with neutral language #1623

Closed
wants to merge 1 commit into from

Conversation

chetanugale
Copy link

@chetanugale chetanugale commented Oct 9, 2021

Fixes #1611

@untergeek
Copy link
Member

Stale. To have this PR considered, the following conditions must be met:

  • Re-open (prefer re-create) against current branches (must also include 6.x and 7.x)
  • Will not accept a PR that results in existing configurations failing. In other words, blacklist must remain acceptable and not break or prevent an existing user's script from functioning after an upgrade. I appreciate what you're doing here, but breaking existing functionality is not acceptable. We can log a deprecation message if blacklist is used, but its presence must not break functionality and usability. We can fully remove in a major version release, but not until it's been deprecated and logged as such for quite a while.
  • Will not accept a PR that doesn't address documentation in both the rST/Sphinx ReadTheDocs documentation as well as the asciidoc docs that render on elastic.co). Change like this must be adequately documented. Cannot just drop this on users without explanation, warning, documentation and examples.

@untergeek untergeek closed this Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replacing non-inclusive term "blacklist" with neutral language
2 participants