Skip to content

Embassy executor spawn token changes.#79

Closed
ericschaal wants to merge 1 commit intoprobe-rs:masterfrom
ericschaal:master
Closed

Embassy executor spawn token changes.#79
ericschaal wants to merge 1 commit intoprobe-rs:masterfrom
ericschaal:master

Conversation

@ericschaal
Copy link
Copy Markdown

@ericschaal ericschaal commented Dec 3, 2025

embassy executor now returns a Result when creating the spawn token, not when spawning.

embassy-rs/embassy@8aec341

@t-moe
Copy link
Copy Markdown
Contributor

t-moe commented Dec 6, 2025

Hey @ericschaal ,
I'm a bit confused by this change. Did embassy_executor publish an API breaking change I'm not aware of?
Can you add some comment?
Thank you

@ericschaal
Copy link
Copy Markdown
Author

Nevermind @t-moe, I got ahead of myself. The breaking change is currently on the master branch but remains unreleased. I'll change the MR to draft, but feel free to close it and I'll resubmit when the change is released by the embassy.

@ericschaal ericschaal marked this pull request as draft December 8, 2025 15:21
@ericschaal ericschaal changed the title Upgrade to embassy executor 0.9.1 Embassy executor spawn token changes. Dec 8, 2025
@bugadani
Copy link
Copy Markdown
Contributor

Embassy-executor 0.10 has been released not too long ago, so this PR could now be merged and released.

Comment thread Cargo.toml
defmt = { version = "1", optional = true }
log = { version = "0.4.20", optional = true }
embassy-executor = { version = "0.9", optional = true, default-features = false }
embassy-executor = { version = "0.9.1", optional = true, default-features = false }
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.

Suggested change
embassy-executor = { version = "0.9.1", optional = true, default-features = false }
embassy-executor = { version = "0.10.0", optional = true, default-features = false }

@t-moe
Copy link
Copy Markdown
Contributor

t-moe commented Mar 27, 2026

Closing this in favor of other PR

@t-moe t-moe closed this Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants