Skip to content

Remove non-inclusive language (#4738)#4743

Merged
lukaszgryglicki merged 10 commits intomainfrom
dev
Aug 6, 2025
Merged

Remove non-inclusive language (#4738)#4743
lukaszgryglicki merged 10 commits intomainfrom
dev

Conversation

@lukaszgryglicki
Copy link
Copy Markdown
Member

Remove non-inclusive language everywhere (about 99%, requested in #4738) except:

  • API parameter names and API URLs (that would require updates in acs-cli, lfx-gateway and all consumers).
  • Dynamo DB table names and dynamo DB item property names (that would require database migration).

cc @mlehotskylf

mlehotskylf and others added 8 commits August 2, 2025 08:42
Signed-off-by: Michal Lehotsky <mlehotsky@linuxfoundation.org>
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io>
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io>
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io>
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io>
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io>
@lukaszgryglicki lukaszgryglicki self-assigned this Aug 6, 2025
Copilot AI review requested due to automatic review settings August 6, 2025 05:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 removes non-inclusive language throughout the codebase by replacing "whitelist" terminology with "allowlist" and updating references from "master" to "main" branch names. The changes maintain backward compatibility by preserving API parameter names, URLs, and DynamoDB table/property names that would require breaking changes across multiple systems.

  • Updates code references, comments, variable names, and function names from "whitelist" to "allowlist"
  • Changes default branch references from "master" to "main"
  • Adds compatibility comments explaining why certain API and database names cannot be changed yet

Reviewed Changes

Copilot reviewed 58 out of 59 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
Various utility scripts Updates script functionality and adds region support
Test files Updates test data and expected values for inclusive language
Documentation files Updates branch references and terminology in docs
Python backend files Comprehensive renaming of classes, functions, and variables
Go backend files Updates struct names, function names, and API handlers
Swagger specifications Updates API operation IDs and model definitions
Configuration files Updates linting rules and workflow triggers

Comment thread cla-backend/cla/controllers/company.py Outdated
Comment thread utils/search_aws_log_group.sh
Comment thread cla-backend/cla/routes.py Outdated
Comment thread cla-backend/cla/routes.py Outdated
Comment thread cla-backend/cla/routes.py Outdated
Comment thread cla-backend/cla/routes.py Outdated
Comment thread cla-backend/cla/routes.py Outdated
mlehotskylf
mlehotskylf previously approved these changes Aug 6, 2025
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io>
@lukaszgryglicki lukaszgryglicki merged commit a12eee6 into main Aug 6, 2025
4 of 5 checks passed
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.

3 participants