Closed didier-wenzek closed 1 week ago
Attention: Patch coverage is 44.44444%
with 5 lines
in your changes missing coverage. Please review.
Project coverage is 78.2%. Comparing base (
5826ae2
) to head (ed0b7fd
). Report is 10 commits behind head on main.
:white_check_mark: Passed | :x: Failed | :next_track_button: Skipped | Total | Pass % | :stopwatch: Duration |
---|---|---|---|---|---|
453 | 0 | 3 | 453 | 100 | 1h1m13.030747999s |
Proposed changes
As rumqttc doesn't trigger a client disconnect when the connection is not established, one has to combine a client disconnect with the use of an
AtomicBool
to break the event loop if for some reason the connection is not established when the user hits ctrl-c.Types of changes
Paste Link to the issue
https://github.com/thin-edge/thin-edge.io/issues/2949
Checklist
cargo fmt
as mentioned in CODING_GUIDELINEScargo clippy
as mentioned in CODING_GUIDELINESFurther comments