Skip to content

build: pin dynamo-llm version in backend-common for cargo publish (1.4.1) - #13670

Merged
pvijayakrish merged 1 commit into
release/1.4.1from
pvijayakrish/backend-common-publish-version-1.4.1
Aug 22, 2026
Merged

pvijayakrish merged 1 commit into
release/1.4.1from
pvijayakrish/backend-common-publish-version-1.4.1

Conversation

@pvijayakrish

@pvijayakrish pvijayakrish commented Aug 22, 2026 •

Copy link
Copy Markdown
Contributor

Cherry-pick of 7fa9245 + 907976f onto release/1.4.1, with the pin stamped to this branch's version.

cargo publish rejects backend-common because its dynamo-llm dependency is a path dep with no version requirement ("all dependencies must have a version requirement specified when publishing" — seen live in the 1.4.1 crates publish; 17 of the 18 crates published, this one cannot). It is a direct path dep because cargo cannot express workspace = true together with default-features = false; pin the version inline (1.4.1, equal to [workspace.package].version) and teach apply_dev_version.py to stamp such inline pins so future version bumps keep it in sync.


Open in Devin Review

…4.1)

Cherry-pick of 7fa9245 + 907976f onto release/1.4.1, with the
pin stamped to this branch's version.

cargo publish rejects backend-common because its dynamo-llm dependency
is a path dep with no version requirement ("all dependencies must have
a version requirement specified when publishing" — seen live in the
1.4.1 crates publish; 17 of the 18 crates published, this one cannot).
It is a direct path dep because cargo cannot express `workspace = true`
together with `default-features = false`; pin the version inline
(1.4.1, equal to [workspace.package].version) and teach
apply_dev_version.py to stamp such inline pins so future version bumps
keep it in sync.

Signed-off-by: pvijayakrish <pvijayakrish@nvidia.com>
@pvijayakrish
pvijayakrish requested review from a team as code owners August 22, 2026 02:53
@github-actions github-actions Bot added the build label Aug 22, 2026

@devin-ai-integration devin-ai-integration Bot 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@pvijayakrish
pvijayakrish merged commit 2112d6b into release/1.4.1 Aug 22, 2026
99 of 111 checks passed
@pvijayakrish
pvijayakrish deleted the pvijayakrish/backend-common-publish-version-1.4.1 branch August 22, 2026 03:56
@pvijayakrish

Copy link
Copy Markdown
Contributor Author

Checks failing are verified to be unrelated and artifactory infra flakiness.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants