Skip to content

Conversation

ctb
Copy link
Contributor

@ctb ctb commented May 12, 2025

Our beta Rust tests are failing with:

error: variables can be used directly in the `format!` string
   --> src/core/src/collection.rs:166:28
    |
166 |                 let path = format!("{}", i);
    |                            ^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
    = note: `-D clippy::uninlined-format-args` implied by `-D warnings`

so I'm seeing if we are ok with fixing these for the rest of the tests.

Copy link

codecov bot commented May 13, 2025

Codecov Report

❌ Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.31%. Comparing base (ead4e87) to head (9fd6548).
⚠️ Report is 67 commits behind head on latest.

Files with missing lines Patch % Lines
src/core/src/ffi/utils.rs 0.00% 1 Missing ⚠️
src/core/src/signature.rs 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           latest    #3640      +/-   ##
==========================================
- Coverage   88.31%   88.31%   -0.01%     
==========================================
  Files         137      137              
  Lines       23469    23468       -1     
  Branches     2270     2270              
==========================================
- Hits        20727    20726       -1     
  Misses       2432     2432              
  Partials      310      310              
Flag Coverage Δ
hypothesis-py 25.44% <ø> (ø)
python 92.62% <ø> (ø)
rust 82.68% <85.71%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@luizirber
Copy link
Member

Or if you want to let clippy fix it, you can run

cargo clippy --fix --all-features -- -Aclippy::all -Wclippy::uninlined_format_args

that's what I did in #3641

@ctb
Copy link
Contributor Author

ctb commented May 14, 2025

Ready for review @luizirber @bluegenes !

@ctb ctb changed the title WIP: try fixing inline variables in rust println! MRG: try fixing inline variables in rust println! May 14, 2025
@ctb ctb merged commit a0ec07c into latest May 14, 2025
42 of 43 checks passed
@ctb ctb deleted the fix_inline_vars branch May 14, 2025 19:12
@ctb ctb mentioned this pull request Jun 1, 2025
ctb added a commit that referenced this pull request Jun 1, 2025
Minor new features:

* add cANI to `sig overlap` (#3644)

Bug fixes:

* make `RevIndex.len()` and `RevIndex.signatures()` use picklist, if set
(#3647)

Cleanup and documentation updates:

* add rocksdb HOWTO (#3648)

Developer updates:

* try fixing inline variables in rust `println!` (#3640)

Dependabot updates:

- Build(deps): Update cachetools requirement from <6,>=4 to >=4,<7
(#3660)
- [pre-commit.ci] pre-commit autoupdate (#3659)
- Build(deps): Bump criterion from 0.5.1 to 0.6.0 (#3655)
- [pre-commit.ci] pre-commit autoupdate (#3654)
- Build(deps): Bump tempfile from 3.19.1 to 3.20.0 (#3639)
- [pre-commit.ci] pre-commit autoupdate (#3638)
ctb added a commit that referenced this pull request Jun 2, 2025
This is a patchfix to v4.9.1
(#3665) to deal with a
botched source code release. Below are the updates since v4.9.0.

Minor new features:

* add cANI to `sig overlap` (#3644)

Bug fixes:

* make `RevIndex.len()` and `RevIndex.signatures()` use picklist, if set
(#3647)

Cleanup and documentation updates:

* add rocksdb HOWTO (#3648)

Developer updates:

* try fixing inline variables in rust `println!` (#3640)

Dependabot updates:

- Build(deps): Update cachetools requirement from <6,>=4 to >=4,<7
(#3660)
- [pre-commit.ci] pre-commit autoupdate (#3659)
- Build(deps): Bump criterion from 0.5.1 to 0.6.0 (#3655)
- [pre-commit.ci] pre-commit autoupdate (#3654)
- Build(deps): Bump tempfile from 3.19.1 to 3.20.0 (#3639)
- [pre-commit.ci] pre-commit autoupdate (#3638)
@ctb ctb mentioned this pull request Jun 28, 2025
ctb added a commit that referenced this pull request Jun 28, 2025
Looks like I forgot to actually release r0.21.0 prior to [release
v4.9.2](#3667), which may
be causing pyodide compile fail here:
pyodide/pyodide-recipes#134

## [0.21.0] - 2025-06-28

MSRV: 1.74

Changes/additions:

* make `RevIndex.len()` and `RevIndex.signatures()` use picklist, if set
(#3647)
* try fixing inline variables in rust `println!` (#3640)

Updates:

* Build(deps): Bump cfg-if from 1.0.0 to 1.0.1 (#3689)
* Build(deps): Bump roaring from 0.10.12 to 0.11.0 (#3702)
* Build(deps): Bump getset from 0.1.5 to 0.1.6 (#3700)
* Build(deps): Bump proptest from 1.6.0 to 1.7.0 (#3674)
* Build(deps): Bump camino from 1.1.9 to 1.1.10 (#3669)
* Build(deps): Bump criterion from 0.5.1 to 0.6.0 (#3655)
* Build(deps): Bump tempfile from 3.19.1 to 3.20.0 (#3639)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants