Skip to content

Conversation

create-pr-actions[bot]
Copy link

Automated update to Github CI workflow cont_integration.yml by create-pull-request GitHub action

@coveralls
Copy link

coveralls commented Jul 15, 2025

Pull Request Test Coverage Report for Build 16281556337

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 84.69%

Totals Coverage Status
Change from base Build 16215822724: 0.0%
Covered Lines: 6577
Relevant Lines: 7766

💛 - Coveralls

notmandatory added a commit that referenced this pull request Jul 31, 2025
d75126e clippy: allow `uninlined_format_args` (valued mammal)
cbf0cfe chore: update rust-version to 1.88.0 (valued mammal)
aa53cc1 clippy: fix uninlined format args (valued mammal)

Pull request description:

  Fix #281 by taking clippy's suggestion to inline the arguments to a `format!` string where applicable.

  Also update `rust-version` to 1.88.0, closing #280.

  ### Notes to the reviewers

  I've gone ahead and fixed the clippy warnings, although some have argued that the lint is quite pedantic and not entirely helpful when it comes to style and readability. Therefore I also allowed the lint to prevent clippy from warning about it in the future d75126e. That change can be reverted if/when rust lang moves `uninlined_format_args` back to the pedantic, i.e. not default group of lints rust-lang/rust-clippy#15287.

  ### Checklists

  #### All Submissions:

  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [ ] ~~This pull request breaks the existing API~~

ACKs for top commit:
  notmandatory:
    ACK d75126e
  oleonardolima:
    ACK d75126e

Tree-SHA512: c34e701fb08f6ada2b0f34c87c93417ca966b021360fa787563f00c6968444028a876bfaf6952cf468f8ad834556b784002d4fc7fd09886fc7420ff1cb6eff9d
@oleonardolima oleonardolima deleted the create-pull-request/update-rust-version branch August 1, 2025 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants