hirosystems / clarinet

Write, test and deploy high-quality smart contracts to the Stacks blockchain and Bitcoin.
https://hiro.so/clarinet
GNU General Public License v3.0
290 stars 129 forks source link

fix: new clippy warning v1.78 #1451

Closed hugocaillard closed 1 month ago

hugocaillard commented 1 month ago

Description

The PR only includes fixes for new clippy checks. All of which seems fine and safe.

This was the most frequent warning in serde/ser.rs https://rust-lang.github.io/rust-clippy/master/index.html#/multiple_bound_locations#%5Bwarn(clippy::multiple_bound_locations)%5D

And a lot of clone_into() or clone_from()

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 23.52941% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 41.16%. Comparing base (b08df15) to head (33f0c3f).

Files Patch % Lines
components/stacks-devnet-js/src/serde/ser.rs 0.00% 22 Missing :warning:
...ents/clarity-lsp/src/common/requests/completion.rs 0.00% 1 Missing :warning:
components/clarity-repl/src/repl/debug/cli.rs 0.00% 1 Missing :warning:
components/clarity-repl/src/repl/session.rs 0.00% 1 Missing :warning:
...clarity-repl/src/test_fixtures/clarity_contract.rs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1451 +/- ## ========================================== + Coverage 41.07% 41.16% +0.08% ========================================== Files 87 87 Lines 32547 32470 -77 ========================================== - Hits 13370 13366 -4 + Misses 19177 19104 -73 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.