UMessageBuilder::build() build UUID if unavailable

eclipse-uprotocol / up-rust

uProtocol Language Specific Library for Rust

Apache License 2.0

11 stars 9 forks source link

Closed PLeVasseur closed 5 months ago

PLeVasseur commented 5 months ago

Adds the ability for UMessageBuilder::build() to make use of UUIDBuilder::build() now that it's a singleton if the message_id was not provide.
Updated doc comments around UMessageBuilder::build() to reflect the distinction.
Highlighted in UMessageBuilder::with_message_id() that typically need not be used anymore, but there may be edge cases.

Fixes #88

PLeVasseur commented 5 months ago

Hey @sophokles73 -- updated based on your feedback :slightly_smiling_face:

PLeVasseur commented 5 months ago

Okay @sophokles73 -- updated based on your feedback.

If you're good with it I think you can squash and merge.

PLeVasseur commented 5 months ago

Okay @sophokles73 -- updated based on your feedback.

If you're good with it I think you can squash and merge.

All right, 2nd attempt.

Already squashed to one commit. If looks good we can merge.