petekubiak
petekubiak
Looks like that works, thanks! So to dive into this a bit deeper, `try_for_each`'s trait requirements are that the "input" is a Result type, hence why the `map(Ok)` is required....
@lulf @igiona We've raised a draft PR #131 for this - it's still WIP but gives an indication of the direction. There's a todo list in the description to indicate...
Following on from that my hope is that you wouldn't even need to call `.add_service()` or `.add_characteristic()` as the attribute macro takes care of that for you when you call...
> Looks great, thanks the work on this! I've left a few inline comments. > > I noticed a commend saying the pub server field would be non-public with additional...
> Looks great, thanks the work on this! I've left a few inline comments. > > I noticed a commend saying the pub server field would be non-public with additional...
> The new integration test seems to be failing, does it pass locally? https://github.com/embassy-rs/trouble/actions/runs/11340899524/job/31538139912 Aha, we don't actually have a rig to run locally (I think I'll try to get...
Thanks for the information, and thank you for adding me! On Tue, 15 Oct 2024, 09:22 Ulf Lilleengen, ***@***.***> wrote: > The new integration test seems to be failing, does...
Would it be possible to add @jamessizeland as well?
/test
Yeah agreed, to me it would seem like it would make more sense for the implementation to live in the serialiser's crate rather than the collection's crate. Good to know...