Skip to content

ci: Use -x instead of -1.0 * x#347

Merged
YeungOnion merged 1 commit intostatrs-dev:masterfrom
FreezyLemon:fix-new-clippy-lints
Jul 25, 2025
Merged

ci: Use -x instead of -1.0 * x#347
YeungOnion merged 1 commit intostatrs-dev:masterfrom
FreezyLemon:fix-new-clippy-lints

Conversation

@FreezyLemon
Copy link
Contributor

This throws clippy warnings in newer Rust versions.

@FreezyLemon
Copy link
Contributor Author

The coverage error seems related, which is surprising. I just used cargo clippy --fix. I'll investigate, this might be about operator precedence

@FreezyLemon FreezyLemon marked this pull request as draft July 20, 2025 09:55
@FreezyLemon FreezyLemon force-pushed the fix-new-clippy-lints branch from e360f9d to 33583af Compare July 20, 2025 10:02
@codecov
Copy link

codecov bot commented Jul 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.99%. Comparing base (2a21f02) to head (33583af).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #347   +/-   ##
=======================================
  Coverage   94.99%   94.99%           
=======================================
  Files          61       61           
  Lines       13606    13606           
=======================================
  Hits        12925    12925           
  Misses        681      681           

☔ 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.

@FreezyLemon
Copy link
Contributor Author

The Windows test failure should be unrelated, as it doesn't use the integral function.

@FreezyLemon FreezyLemon marked this pull request as ready for review July 20, 2025 19:51
@YeungOnion
Copy link
Contributor

Think this lint was a barrier to seeing when that test started failing on windows, this will help!

@YeungOnion YeungOnion merged commit 8a45d47 into statrs-dev:master Jul 25, 2025
8 of 10 checks passed
@FreezyLemon FreezyLemon deleted the fix-new-clippy-lints branch July 26, 2025 08:20
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