Skip to content

Canonicalize Pillow dependency constraint#2194

Closed
SharonHart wants to merge 1 commit into
mainfrom
fix/canonicalize-pillow-requirement
Closed

Canonicalize Pillow dependency constraint#2194
SharonHart wants to merge 1 commit into
mainfrom
fix/canonicalize-pillow-requirement

Conversation

@SharonHart

Copy link
Copy Markdown
Contributor

Summary

  • express the image redactor Pillow requirement in canonical PEP 508 form
  • preserve the existing >=12.3.0,<13.0.0 version range exactly
  • force GitHub dependency graph/SBOM to reparse the requirement after Raise Pillow minimum to 12.3.0 #2188

This is a dependency-graph reparsing workaround; it does not change the resolved Pillow version or any unrelated dependency. uv.lock was regenerated with uv 0.11.6 and remained unchanged.

Validation

  • poetry check
  • /tmp/presidio-uv-0116/bin/uv lock
  • /tmp/presidio-uv-0116/bin/uv lock --check
  • git diff --check

Use canonical PEP 508 syntax to force GitHub's dependency graph to reparse the updated Pillow range.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: e2c6468b-1ee0-4d4b-bf77-923dea13a420
Copilot AI review requested due to automatic review settings July 23, 2026 20:55
@github-actions

Copy link
Copy Markdown
Contributor

Coverage report (presidio-anonymizer)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  presidio-anonymizer/presidio_anonymizer
  __init__.py
  anonymizer_engine.py
  presidio-anonymizer/presidio_anonymizer/operators
  custom.py
Project Total  

This report was generated by python-coverage-comment-action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage report (presidio-structured)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  presidio-structured/presidio_structured/data
  data_processors.py
Project Total  

This report was generated by python-coverage-comment-action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage report (presidio-cli)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  presidio-cli/presidio_cli
  cli.py
Project Total  

This report was generated by python-coverage-comment-action

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

Updates presidio-image-redactor’s Pillow dependency declaration to a canonical PEP 508 requirement string to trigger GitHub dependency graph/SBOM reparsing (without changing the effective version constraints).

Changes:

  • Rewrites the Pillow requirement from pillow (>=12.3.0,<13.0.0) to pillow>=12.3.0,<13.0.0 while preserving the exact version range.

@github-actions

Copy link
Copy Markdown
Contributor

Coverage report (presidio-image-redactor)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  presidio-image-redactor/presidio_image_redactor
  dicom_image_pii_verify_engine.py
  document_intelligence_ocr.py
  image_analyzer_engine.py
Project Total  

This report was generated by python-coverage-comment-action

@github-actions

Copy link
Copy Markdown
Contributor

Coverage report (presidio-analyzer)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  presidio-analyzer/presidio_analyzer
  analyzer_engine.py
  entity_recognizer.py
  presidio-analyzer/presidio_analyzer/context_aware_enhancers
  lemma_context_aware_enhancer.py
  presidio-analyzer/presidio_analyzer/input_validation
  schemas.py
  yaml_recognizer_models.py
  presidio-analyzer/presidio_analyzer/nlp_engine
  __init__.py
  nlp_engine_provider.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers
  __init__.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/country_specific
  __init__.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/country_specific/finland
  fi_personal_identity_code_recognizer.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/country_specific/germany
  de_bsnr_recognizer.py
  de_id_card_recognizer.py
  de_lanr_recognizer.py
  de_passport_recognizer.py
  de_social_security_recognizer.py
  de_tax_id_recognizer.py
  de_vat_id_recognizer.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/country_specific/india
  in_vehicle_registration_recognizer.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/country_specific/poland
  pl_pesel_recognizer.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/country_specific/singapore
  sg_uen_recognizer.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/country_specific/us
  us_ssn_recognizer.py
  presidio-analyzer/presidio_analyzer/predefined_recognizers/generic
  phone_recognizer.py
  presidio-analyzer/presidio_analyzer/recognizer_registry
  recognizer_registry.py
  recognizer_registry_provider.py
  recognizers_loader_utils.py
Project Total  

The report is truncated to 25 files out of 72. To see the full report, please visit the workflow summary page.

This report was generated by python-coverage-comment-action

@SharonHart

Copy link
Copy Markdown
Contributor Author

Closing as a duplicate of #2193, which contains the same focused PEP 508 syntax-only change and validation.

@SharonHart SharonHart closed this Jul 24, 2026
@SharonHart
SharonHart deleted the fix/canonicalize-pillow-requirement branch July 24, 2026 06:54
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