Update embedded-services workspace deps to main branch#99
Merged
tullom merged 1 commit intoJun 4, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the EC workspace’s embedded-services git dependencies for time-alarm-service-messages and battery-service-messages to track the main branch instead of v0.2.0, aligning the workspace with upstream embedded-services development.
Changes:
- Switched
time-alarm-service-messagesworkspace dependency frombranch = "v0.2.0"tobranch = "main". - Switched
battery-service-messagesworkspace dependency frombranch = "v0.2.0"tobranch = "main".
Cargo Vet Audit Passed
|
tullom
force-pushed
the
update-embedded-services-main
branch
from
ec67ea9 to
a3195a7
Compare
kurtjd
previously approved these changes
Jun 2, 2026
tullom
force-pushed
the
update-embedded-services-main
branch
from
a3195a7 to
f3e25f5
Compare
jerrysxie
previously approved these changes
Jun 2, 2026
tullom
force-pushed
the
update-embedded-services-main
branch
2 times, most recently
from
9d3e88c to
9993d22
Compare
The battery-service-messages, time-alarm-service-messages, and thermal-service-messages crates no longer exist on the main branch of embedded-services. They have been split into separate *-service-interface (data types and traits) and *-service-relay (MCTP request/response message enums) crates. Switch workspace and crate dependencies accordingly, update imports across test-cli, test-lib, test-script, and test-tui, rename AcpiBatteryRequest/Response variants to their shorter forms (e.g. BatteryGetBstRequest -> GetBst), wrap the thermal temperature field unwrap with .0 for the new DeciKelvin type, and switch BIX enum decoding to the standard TryFrom impls now that the bat_*/power_unit_try_from_u32 helpers are gone. Pin embedded-mcu-hal to the published 0.2.0 release on crates.io to match the version that embedded-services pulls in transitively, and update the embedded-mcu-hal API usages (UncheckedDatetime -> DatetimeFields, to_unix_time_seconds -> unix_timestamp). Assisted-by: GitHub Copilot:claude-opus-4.7
tullom
force-pushed
the
update-embedded-services-main
branch
from
9993d22 to
32db859
Compare
kurtjd
approved these changes
Jun 3, 2026
jerrysxie
approved these changes
Jun 4, 2026
felipebalbi
approved these changes
Jun 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The battery-service-messages, time-alarm-service-messages, and thermal-service-messages crates no longer exist on the main branch of embedded-services. They have been split into separate *-service-interface (data types and traits) and *-service-relay (MCTP request/response message enums) crates.
Switch workspace and crate dependencies accordingly, update imports across test-cli, test-lib, test-script, and test-tui, rename AcpiBatteryRequest/Response variants to their shorter forms (e.g. BatteryGetBstRequest -> GetBst), wrap the thermal temperature field unwrap with .0 for the new DeciKelvin type, and switch BIX enum decoding to the standard TryFrom impls now that the bat_*/power_unit_try_from_u32 helpers are gone.
Pin embedded-mcu-hal to the published 0.2.0 release on crates.io to match the version that embedded-services pulls in transitively, and update the embedded-mcu-hal API usages (UncheckedDatetime -> DatetimeFields, to_unix_time_seconds -> unix_timestamp).
Assisted-by: GitHub Copilot:claude-opus-4.7