Skip to content

Fix/146 pii scrubber phone number - #1017

Open
a-maryam wants to merge 8 commits into
ascherj:mainfrom
a-maryam:fix/146-pii-scrubber-phone-number
Open

Fix/146 pii scrubber phone number#1017
a-maryam wants to merge 8 commits into
ascherj:mainfrom
a-maryam:fix/146-pii-scrubber-phone-number

Conversation

@a-maryam

@a-maryam a-maryam commented Aug 9, 2026

Copy link
Copy Markdown

Summary

This PR updates the regex in pii_scrubber.py, so that phone numbers are correctly redacted. It includes associated tests

Issue

Closes #146

Changes

  • Updated regex in pii_scrubber.py
  • Wrote tests for redaction code

Testing

  • [ X ] Unit tests pass (make test-unit): Pass
  • [ X ] Integration tests pass (make test-integration): Pass
  • [ X ] Linter passes (make lint): Pass
  • [ X ] Type checker passes (make typecheck): Pass
  • [ X ] New/updated tests cover the changes: Pass

Screenshots / Demo

Notes for Reviewers

Pre-existing linting/pre-commit hook errors in pii_scrubber.py

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.

PII scrubber fails to redact parenthesized US phone numbers

1 participant