Skip to content

Revert "Remove non-inclusive language (#4738)"#4744

Closed
lukaszgryglicki wants to merge 1 commit into
mainfrom
revert-4743-dev
Closed

Revert "Remove non-inclusive language (#4738)"#4744
lukaszgryglicki wants to merge 1 commit into
mainfrom
revert-4743-dev

Conversation

@lukaszgryglicki

Copy link
Copy Markdown
Member

This is just in case anything goes wrong...

Do NOT merge this.

Reverts #4743

cc @mlehotskylf

@lukaszgryglicki lukaszgryglicki self-assigned this Aug 6, 2025
Copilot AI review requested due to automatic review settings August 6, 2025 06:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reverts changes made in the "Remove non-inclusive language" pull request, essentially restoring "whitelist" terminology throughout the codebase instead of "allowlist" and related non-inclusive language changes. The PR description indicates this is a precautionary revert and should not be merged.

  • Reverts terminology changes from "allowlist/approval list" back to "whitelist" across code, comments, API endpoints, and documentation
  • Reverts Git branch references from "main" back to "master" in various configuration files and documentation
  • Reverts error terminology changes from "denylist" back to "blacklist" in configuration files

Reviewed Changes

Copilot reviewed 58 out of 59 changed files in this pull request and generated 1 comment.

File Description
Multiple Python files Restore "whitelist" terminology in method names, variables, comments, and API parameters
Multiple Go files Revert API handlers, models, and variable names from allowlist back to whitelist terminology
Documentation files Restore "master" branch references and whitelist terminology in markdown files
Configuration files Revert workflow configurations to use "master" branch and restore "blacklist" terminology

Comment thread cla-backend/cla/controllers/company.py
@lukaszgryglicki

Copy link
Copy Markdown
Member Author

Not needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants