Update Rust crate temporalio-common to c09a290#8421
Update Rust crate temporalio-common to c09a290#8421hash-worker[bot] wants to merge 1 commit intomainfrom
temporalio-common to c09a290#8421Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Regenerates Written by Cursor Bugbot for commit b191eea. This will update automatically on new commits. Configure here. |
🤖 Augment PR SummarySummary: This PR updates the pinned git revision for the Rust crate Changes:
Technical Notes: Since both crates come from the same upstream repository, mixing revisions may result in multiple git checkouts/duplicate crate versions unless they are intentionally kept independent. 🤖 Was this summary useful? React with 👍 or 👎 |
| temporalio-client = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" } | ||
| temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" } | ||
| temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "fb14d9c3cf4e9e6d0894fd9bb2985dbf6d6de1fe" } | ||
| test-log = { version = "0.2.18", default-features = false } |
There was a problem hiding this comment.
temporalio-common is now pinned to a different git rev than temporalio-client (same upstream repo), which can cause Cargo to pull two distinct copies and potentially lead to type/version mismatches. Is it expected that these two crates are compatible across different commits, or should they be bumped together to the same rev?
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8421 +/- ##
==========================================
- Coverage 62.70% 62.70% -0.01%
==========================================
Files 1302 1302
Lines 131589 131595 +6
Branches 5487 5487
==========================================
Hits 82512 82512
- Misses 48169 48175 +6
Partials 908 908
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Merging this PR will not alter performance
Comparing Footnotes
|
58f81bb to
b8b4e23
Compare
temporalio-common to fb14d9ctemporalio-common to 20a6817
temporalio-common to 20a6817temporalio-common to 12c656f
b8b4e23 to
b67cdc2
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Cargo.toml
Outdated
| tarpc = { version = "0.36.0", default-features = false, git = "https://github.com/google/tarpc", rev = "f55f36d2d876b1868cfcf52f41d0456a60cf726c" } | ||
| temporalio-client = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" } | ||
| temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" } | ||
| temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "12c656f7b2dce2a139bb3f1b4873c7007a751380" } |
There was a problem hiding this comment.
Temporal SDK revisions are now inconsistent
Medium Severity
temporalio-client stays on rev 231e21c while temporalio-common is bumped to 12c656f..., so the workspace now resolves two different temporalio-common crates. This can break interoperability where hash-temporal-client uses both crates, because shared SDK types may no longer be identical across crate boundaries.
Additional Locations (1)
temporalio-common to 12c656ftemporalio-common to f16f160
9ff8edf to
34cbc3a
Compare
temporalio-common to 78a94detemporalio-common to 7161ed3
34cbc3a to
f00e489
Compare
temporalio-common to 7161ed3temporalio-common to 37d8c1a
f00e489 to
63b5f68
Compare
temporalio-common to 37d8c1atemporalio-common to 8f01a38
63b5f68 to
0979281
Compare
0979281 to
b191eea
Compare
temporalio-common to 8f01a38temporalio-common to c09a290


This PR contains the following updates:
231e21c->c09a290Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Configuration
📅 Schedule: Branch creation - "before 4am every weekday,every weekend" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Renovate Bot.