Skip to content

Conversation

@magravel
Copy link

@magravel magravel commented Jun 10, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Jun 10, 2025

Cargo Vet Audit Passed

cargo vet has passed in this PR. No new unvetted dependencies were found.

@github-actions github-actions bot added the cargo vet PRs pending auditor review label Jun 10, 2025
@magravel magravel force-pushed the cfu-trait-refactoring-integration branch from 6ac6d9f to 2a3e8bd Compare June 26, 2025 17:18
@magravel magravel force-pushed the cfu-trait-refactoring-integration branch 3 times, most recently from c37b073 to 332ca06 Compare July 7, 2025 22:34
@magravel magravel force-pushed the cfu-trait-refactoring-integration branch from 332ca06 to 83c00e9 Compare July 7, 2025 23:45
kurtjd pushed a commit that referenced this pull request Jul 8, 2025
… and update examples (#386)

Thank you to @magravel for fixing the changes in embedded-cfu-protocol.
This PR includes the same .rs changes in #341.

- Update cfu-service to use constructs in embedded-cfu-protocol v0.2.0
- Further testing showed that I missed some Cells in
battery-service/device.rs that should've been fixed in #372
    - Fixed in this PR
- Update examples to be Sync safe and patch embedded-services to remove
a circular dependency in the examples' `Cargo.toml`
@jeffglaum jeffglaum moved this to In review in Embedded Controller Oct 1, 2025
@jeffglaum jeffglaum added this to the 9. Code Quality & Testing milestone Nov 17, 2025
@jeffglaum jeffglaum assigned RobertZ2011 and unassigned magravel Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cargo vet PRs pending auditor review

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

3 participants