Skip to content

Canonicalize Pillow requirement syntax#2195

Open
SharonHart wants to merge 1 commit into
mainfrom
fix-pillow-pep508-reparse
Open

Canonicalize Pillow requirement syntax#2195
SharonHart wants to merge 1 commit into
mainfrom
fix-pillow-pep508-reparse

Conversation

@SharonHart

Copy link
Copy Markdown
Contributor

Change Description

Canonicalize the image redactor Pillow requirement to pillow>=12.3.0,<13.0.0 without changing its version range. This is a GitHub dependency-graph/SBOM reparsing workaround after #2188: removing the optional PEP 508 parentheses prompts GitHub to reparse the source requirement and discard the stale duplicate Pillow 12.2.0 entry.

uv.lock was regenerated with uv 0.11.6 and remained unchanged. This replaces #2193 solely to use the required branch name.

Issue reference

Follow-up to #2188.

Checklist

  • I have reviewed the contribution guidelines
  • I agree to follow the project Code of Conduct
  • I confirm that I have the right to submit this contribution and that it does not knowingly contain proprietary or confidential code.
  • Unit tests are not applicable to this metadata-only syntax change
  • Metadata and lock checks pass locally
  • Documentation updates are not required

Validation:

  • poetry check
  • uv tool run --from uv==0.11.6 uv lock --check
  • git diff --check

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 24, 2026 07:47
@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

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

This PR updates the presidio-image-redactor package metadata to canonicalize the Pillow dependency specifier (removing optional PEP 508 parentheses) while keeping the same version range, to trigger GitHub dependency-graph/SBOM reparsing and eliminate a stale duplicated entry.

Changes:

  • Update Pillow requirement string from pillow (>=12.3.0,<13.0.0) to pillow>=12.3.0,<13.0.0 (no version-range change).

@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-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-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

@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 SharonHart closed this Jul 24, 2026
@SharonHart
SharonHart deleted the fix-pillow-pep508-reparse branch July 24, 2026 08:14
@SharonHart

Copy link
Copy Markdown
Contributor Author

Closing as a duplicate of #2193. The alternate branch name was not a user requirement, and #2193 already contains the validated one-file change.

@SharonHart
SharonHart restored the fix-pillow-pep508-reparse branch July 24, 2026 12:27
@SharonHart SharonHart reopened this Jul 24, 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