Skip to content

Conversation

@brainstorm
Copy link
Contributor

No description provided.

@brainstorm brainstorm force-pushed the heapless_embedded-io-async_defmt_version_bump branch from 4b66f3b to 1487feb Compare November 27, 2025 05:31
@Dirbaio
Copy link
Member

Dirbaio commented Nov 27, 2025

See #4745 for embedded-io

We can't upgade heapless until smoltcp upgrades it.

About defmt: we're already using 1.0 everywhere, there's nothing to upgrade.

@brainstorm
Copy link
Contributor Author

See #4745 for embedded-io

We can't upgade heapless until smoltcp upgrades it.

About defmt: we're already using 1.0 everywhere, there's nothing to upgrade.

Thanks for the feedback @Dirbaio! ... what's the blocker on heapless 0.9 for smol? I see it merged already in smoltcp-rs/smoltcp#1083 ?

@Dirbaio
Copy link
Member

Dirbaio commented Nov 27, 2025

A release.

@brainstorm
Copy link
Contributor Author

brainstorm commented Nov 30, 2025

About defmt: we're already using 1.0 everywhere, there's nothing to upgrade.

If I understood correctly from the last comment in rust-embedded/embedded-hal#716 (comment) the removal of all defmt-03 references is a good cleanup to do/change (upgrade) regardless? ... and if it's still breaking, would it make sense to semver-bump it accordingly or you want to keep defmt-03 for a while longer?

A [smol] release.

Pre-xmas communitly present? 🙏🏻 Let me know if I can help in any way with this :)

I'll now get acquainted with https://github.com/embassy-rs/cargo-embassy-devtool to compile all embassy-rs locally avoid CI failed builds spam (sorry about them!).

@Dirbaio
Copy link
Member

Dirbaio commented Nov 30, 2025

the removal of all defmt-03 references is a good cleanup to do/change (upgrade) regardless?

Embassy crates don't define any defmt-03 features, they just use them from dependencies. If a new version of a dep adds the defmt feature we should upgrade to it, of course.

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.

2 participants