Skip to content

fix(dev): validate integration version bumps - #8139

Merged
Xuanwo merged 2 commits into
mainfrom
xuanwo/validate-integration-version-bumps
Aug 26, 2026
Merged

fix(dev): validate integration version bumps#8139
Xuanwo merged 2 commits into
mainfrom
xuanwo/validate-integration-version-bumps

Conversation

@Xuanwo

@Xuanwo Xuanwo commented Aug 25, 2026

Copy link
Copy Markdown
Member

Which issue does this PR close?

Follow-up to #8008.

Rationale for this change

object_store_opendal 0.58.1 upgraded object_store from 0.13 to 0.14 under a patch release, which broke downstream trait compatibility. The release tool currently trusts manually selected package versions and can repeat this mistake when an integration exposes types or traits from an incompatible dependency line.

What changes are included in this PR?

Before writing any files, update-version now compares configured public compatibility dependencies with the latest reachable final release tag. It requires object_store_opendal and parquet_opendal to cross the corresponding Cargo compatibility boundary when opendal, object_store, or parquet does.

The check uses the planned OpenDAL version from the release package graph and external dependency versions from the integration manifests. Missing release tags and unsupported dependency requirement shapes fail with actionable errors.

Are there any user-facing changes?

Release managers receive an error before version files are updated when an integration target is incompatible. Library APIs and runtime behavior do not change.

AI Usage Statement

Codex materially assisted with repository and release-history analysis, implementation, and regression-test design. The change assumes the latest reachable non-prerelease tag represents the release baseline and accepts dependency requirements with one caret, exact, or tilde comparator; other requirement shapes fail closed.

@Xuanwo
Xuanwo marked this pull request as ready for review August 25, 2026 09:17
@Xuanwo
Xuanwo requested a review from tisonkun as a code owner August 25, 2026 09:17
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Aug 25, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 26, 2026
@Xuanwo
Xuanwo merged commit b2d1764 into main Aug 26, 2026
38 checks passed
@Xuanwo
Xuanwo deleted the xuanwo/validate-integration-version-bumps branch August 26, 2026 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants