Skip to content

Use inclusive allowlist and denylist terminology - #27

Merged
berezovskyi merged 3 commits into
masterfrom
inclusive-list-terminology
Sep 30, 2026
Merged

berezovskyi merged 3 commits into
masterfrom
inclusive-list-terminology

Conversation

@berezovskyi

@berezovskyi berezovskyi commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Replace whitelist/blacklist terminology in the tooling with allowlist/denylist.
  • Rename the namespace helper class and update all its callers.
  • Add a GitHub Actions Maven test check for tools/tools-common, the module changed here.

Validation

  • git diff --check passes and the workflow YAML parses successfully.
  • The targeted Maven CI check passes.
  • A broader Maven sweep reached tools/examplechecker, where three fixture-output assertions fail; that module is outside this change.

@berezovskyi
berezovskyi marked this pull request as ready for review September 30, 2026 17:43
@berezovskyi
berezovskyi marked this pull request as draft September 30, 2026 17:47
@berezovskyi
berezovskyi marked this pull request as ready for review September 30, 2026 17:54
@berezovskyi
berezovskyi requested a balanced review from Copilot September 30, 2026 17:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@berezovskyi
berezovskyi merged commit d98224c into master Sep 30, 2026
4 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