Skip to content

Conversation

@qlrd
Copy link
Contributor

@qlrd qlrd commented May 17, 2025

What is the purpose of this pull request?

  • Bug fix
  • Documentation update
  • New feature
  • Test
  • Other: lint.

Which crates are being modified?

  • floresta-chain
  • floresta-cli
  • floresta-common
  • floresta-compact-filters
  • floresta-electrum
  • floresta-watch-only
  • floresta-wire
  • floresta
  • florestad
  • Other:

Description

A new clippy warning was raised for cloned-ref-to-slice. This commit fixes the warning that raise an error during lint-features just recipe.

Notes to the reviewers

During a CI job in #465, a new clippy warning was raised, maybe due to nightly specs for clippy.

Author Checklist

  • I've followed the contribution guidelines
  • I've verified one of the following:
    • Ran just pcc (recommended but slower)
    • Ran just lint-features '-- -D warnings' && cargo test --release
    • Confirmed CI passed on my fork
  • I've linked any related issue(s) in the sections above

@Davidson-Souza Davidson-Souza added the lint This issue or PR relates to code style and linting label May 17, 2025
A new clippy warning was raised for cloned-ref-to-slice. This commit
fixes the warning that raise an error during lint-features just recipe.
@qlrd qlrd force-pushed the fix-clippy-slice-descriptor branch from d9a0e67 to f4af890 Compare May 17, 2025 19:12
@qlrd qlrd requested a review from Davidson-Souza May 17, 2025 20:17
Copy link
Member

@Davidson-Souza Davidson-Souza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK f4af890

@Davidson-Souza Davidson-Souza merged commit da75f24 into vinteumorg:master May 17, 2025
10 checks passed
@qlrd qlrd deleted the fix-clippy-slice-descriptor branch May 17, 2025 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lint This issue or PR relates to code style and linting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants