docs: add async SingleLatest example#112
Conversation
lxsaah
left a comment
There was a problem hiding this comment.
Review: PR #112 — async SingleLatest example
The PR itself is correct and well-scoped — verified locally: cargo build / cargo run (stdout matches the README), clippy -D warnings, fmt, and git diff --check all clean; API usage cross-checked against aimdb-core / aimdb-tokio-adapter; workspace wiring (Cargo.toml, Makefile, Cargo.lock, README) is complete.
Two minor nits below, each with a one-click committable suggestion. Both are non-blocking in substance — flagged as request-changes so they're resolved before merge.
Signed-off-by: Phung <nvphungdev@users.noreply.github.com>
|
Thanks for the review. I applied both suggestions: made the Mailbox link relative and added the SingleLatest stdout note for overwritten early rollout values.\n\nChecked with |
lxsaah
left a comment
There was a problem hiding this comment.
Thanks a lot for this contribution @nvphungdev and for turning around the review feedback so quickly. Both items are resolved cleanly and I've re-verified locally.
Merging now, feel free to pick up another issue from the list. Happy to have you working on more.
Summary
examples/hello-single-latest-asyncwith an async.source()/.tap()demo forBufferCfg::SingleLatest.Closes #100
Validation
cargo run -p hello-single-latest-asynccargo fmt -p hello-single-latest-async -- --checkcargo clippy -p hello-single-latest-async -- -D warningsmake fmt-checkgit diff --checkNot run locally
make check/make all: this local machine does not have thethumbv7em-none-eabihftarget installed, and those targets also cover the full embedded/security workspace outside this docs/example change.