Closed Ruadhri17 closed 4 months ago
Attention: Patch coverage is 92.30769%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 78.1%. Comparing base (
8a91efb
) to head (05a644e
). Report is 6 commits behind head on main.
:white_check_mark: Passed | :x: Failed | :next_track_button: Skipped | Total | Pass % | :stopwatch: Duration |
---|---|---|---|---|---|
442 | 0 | 3 | 442 | 100 | 1h1m19.518955999s |
Proposed changes
This PR adds an extra check in
software_update
to skip the download process if the payload contains an empty url. A system test was added to prove that the error no longer exists when an empty url is provided in the local command.Types of changes
Paste Link to the issue
2857
Checklist
cargo fmt
as mentioned in CODING_GUIDELINEScargo clippy
as mentioned in CODING_GUIDELINESFurther comments