Skip to content

Fix eachdist dependency version matching#5237

Open
HasanTasdiq wants to merge 2 commits into
open-telemetry:mainfrom
HasanTasdiq:codex/fix-eachdist-dependency-match
Open

Fix eachdist dependency version matching#5237
HasanTasdiq wants to merge 2 commits into
open-telemetry:mainfrom
HasanTasdiq:codex/fix-eachdist-dependency-match

Conversation

@HasanTasdiq
Copy link
Copy Markdown

@HasanTasdiq HasanTasdiq commented May 20, 2026

Fixes #5227

Summary

  • Tightened the dependency version replacement regex in scripts/eachdist.py.
  • Prevented package names that are prefixes of another package, such as opentelemetry-proto, from matching opentelemetry-proto-json.
  • Added the towncrier changelog fragment for this release-tooling fix.

Why

The release helper matched package name prefixes too broadly, so updating opentelemetry-proto could also rewrite dependencies for opentelemetry-proto-json.

Testing

  • Ran: targeted Python validation snippet for update_dependencies and update_patch_dependencies using temporary pyproject.toml fixtures - passed
  • Ran: python3 -m py_compile scripts/eachdist.py - passed

Notes

  • Full tox/precommit not run locally because uv and tox are not installed in this shell.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented May 20, 2026

CLA Signed
The committers listed above are authorized under a signed CLA.

@HasanTasdiq HasanTasdiq marked this pull request as ready for review May 20, 2026 22:52
@HasanTasdiq HasanTasdiq requested a review from a team as a code owner May 20, 2026 22:52
Copy link
Copy Markdown
Contributor

@xrmx xrmx left a comment

Choose a reason for hiding this comment

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

I think the regexp can be simplified by assuming there's whitespace between package name and version.

@github-project-automation github-project-automation Bot moved this to Reviewed PRs that need fixes in Python PR digest May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Reviewed PRs that need fixes

Development

Successfully merging this pull request may close these issues.

scripts/eachdist.py dependency version search is not strict enough

2 participants