Skip to content

Add ISO 8601 support to DateRecognizer#1621

Merged
omri374 merged 6 commits into
microsoft:mainfrom
StefH:stef-DateRecognizer-ISO8601
Jun 9, 2025
Merged

Add ISO 8601 support to DateRecognizer#1621
omri374 merged 6 commits into
microsoft:mainfrom
StefH:stef-DateRecognizer-ISO8601

Conversation

@StefH

@StefH StefH commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

Change Description

Added regular expression to validate a ISO 8601 date

Issue reference

This PR fixes issue #1619

Checklist

  • I have reviewed the contribution guidelines
  • I have signed the CLA (if required)
  • My code includes unit tests
  • All unit tests and lint checks pass locally
  • My PR contains documentation updates / additions if required

Comment thread presidio-analyzer/tests/test_date_recognizer.py
Comment thread presidio-analyzer/presidio_analyzer/predefined_recognizers/date_recognizer.py Outdated
@omri374

omri374 commented Jun 6, 2025

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines

Copy link
Copy Markdown
Commenter does not have sufficient privileges for PR 1621 in repo microsoft/presidio

@omri374

omri374 commented Jun 8, 2025

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

omri374
omri374 previously approved these changes Jun 8, 2025
@omri374

omri374 commented Jun 8, 2025

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@StefH

StefH commented Jun 9, 2025

Copy link
Copy Markdown
Contributor Author

@omri374
I did fix the unit tests; can you start the pipeline again?

@omri374

omri374 commented Jun 9, 2025

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@omri374 omri374 merged commit 14cc3c8 into microsoft:main Jun 9, 2025
34 checks passed
prokopidis pushed a commit to prokopidis/presidio that referenced this pull request Jun 23, 2026
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.

2 participants