Skip to content

Conversation

@jerrysxie
Copy link
Contributor

No description provided.

@jerrysxie jerrysxie self-assigned this Aug 29, 2025
@jerrysxie jerrysxie added the dependencies Dependencies have been updated label Aug 29, 2025
@github-actions
Copy link

github-actions bot commented Aug 29, 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 Aug 29, 2025
JamesHuard
JamesHuard previously approved these changes Aug 29, 2025
@jeffglaum jeffglaum moved this to In review in Embedded Controller Aug 29, 2025
asasine
asasine previously approved these changes Aug 29, 2025
felipebalbi
felipebalbi previously approved these changes Aug 29, 2025
@jerrysxie jerrysxie dismissed stale reviews from felipebalbi, asasine, and JamesHuard via faa1222 August 29, 2025 16:07
@jerrysxie jerrysxie marked this pull request as ready for review August 29, 2025 16:08
Copilot AI review requested due to automatic review settings August 29, 2025 16:08
@jerrysxie jerrysxie requested a review from a team as a code owner August 29, 2025 16:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR migrates embassy crates from git dependencies to their latest published versions on crates.io. This is a dependency update that standardizes the project to use stable, published releases instead of git references.

  • Replaces git dependencies with version-specific crates.io dependencies for all embassy crates
  • Updates to the latest available versions (embassy-sync 0.7.2, embassy-executor 0.9.0, etc.)
  • Maintains all existing feature configurations

Reviewed Changes

Copilot reviewed 4 out of 8 changed files in this pull request and generated no comments.

File Description
Cargo.toml Updates workspace-level embassy dependencies from git to crates.io versions
examples/std/Cargo.toml Migrates embassy dependencies for the std example
examples/rt685s-evk/Cargo.toml Migrates embassy dependencies for the rt685s-evk example
examples/rt633/Cargo.toml Migrates embassy dependencies for the rt633 example

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jerrysxie jerrysxie requested a review from a team as a code owner August 29, 2025 16:12
@jerrysxie jerrysxie added the BREAKING CHANGE Marks breaking changes label Aug 29, 2025
@jerrysxie jerrysxie enabled auto-merge (squash) August 29, 2025 16:19
@jerrysxie jerrysxie merged commit b30b004 into OpenDevicePartnership:main Aug 29, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Embedded Controller Aug 29, 2025
Comment on lines 555 to 579
[[package]]
name = "embassy-executor"
version = "0.8.0"
source = "git+https://github.com/embassy-rs/embassy#ef673c6ca310cf0a7e9b1254afb7806bd6879e94"
version = "0.7.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "90327bcc66333a507f89ecc4e2d911b265c45f5c9bc241f98eee076752d35ac6"
dependencies = [
"critical-section",
"document-features",
"embassy-executor-macros 0.6.2",
]

[[package]]
name = "embassy-executor"
version = "0.9.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "d0c80c92a31c7f6b02a938f10feea17ea3cc0e8d33bcac7f3fe8cede3723bb56"
dependencies = [
"cortex-m",
"critical-section",
"defmt 1.0.1",
"document-features",
"embassy-executor-macros",
"embassy-executor-macros 0.7.0",
"embassy-executor-timer-queue",
"log",
]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we sort out the double embassy-executor dependency?

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

Labels

BREAKING CHANGE Marks breaking changes cargo vet PRs pending auditor review dependencies Dependencies have been updated

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants