Skip to content

Add minimal example: hello-mailbox-async#151

Open
CernyMil wants to merge 2 commits into
aimdb-dev:mainfrom
CernyMil:Add-minimal-example--hello-mailbox-async--#99
Open

Add minimal example: hello-mailbox-async#151
CernyMil wants to merge 2 commits into
aimdb-dev:mainfrom
CernyMil:Add-minimal-example--hello-mailbox-async--#99

Conversation

@CernyMil

Copy link
Copy Markdown

Description

A clear and concise description of the changes.

Related Issue

  • Closes # (if applicable)

Checklist

  • I have read the CONTRIBUTING.md document.
  • My code follows the project's coding standards.
  • I have added tests to cover my changes.
  • All new and existing tests passed (make check).
  • I have updated the documentation accordingly.

@CernyMil CernyMil requested a review from lxsaah as a code owner June 28, 2026 20:03

@lxsaah lxsaah left a comment

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.

Thanks @CernyMil. The example is clear and the Mailbox latest-wins demo works as intended. I checked it out locally and cargo run -p hello-mailbox-async produces the expected Got: Blue output. A few inline comments before this can merge. The blocker is the embassy = "0.0.0" line in aimdb-core/Cargo.toml: that's the cause of the could not compile embassy errors from #99 and it will fail the embedded/wasm CI matrix here, so it needs to be removed. Beyond that, please move the in-closure sleeps to the runtime timer and finish the Makefile wiring from the issue checklist. Once those are addressed this should be ready.

Comment thread aimdb-core/Cargo.toml Outdated
Comment thread examples/hello-mailbox-async/src/main.rs Outdated
Comment thread Cargo.toml
Comment thread examples/hello-mailbox-async/README.md
Comment thread Cargo.toml
@CernyMil CernyMil requested a review from lxsaah June 29, 2026 05:47
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