Skip to content

Tighten up clippy #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Jun 27, 2025
Merged

Tighten up clippy #22

merged 16 commits into from
Jun 27, 2025

Conversation

LebedevRI
Copy link
Member

No description provided.

Copy link

The proposed diff is not rustfmtted.
To make this check pass, download the following patch
(via browser, you must be logged-in in order for this URL to work),
(NOTE: save it into the repo checkout dir for the snippet to work)
https://github.com/darktable-org/rawspeed.rs/actions/runs/15937233509/artifacts/3422859902
... and run:

cd <path/to/repo/checkout> # NOTE: use your own path here
unzip rustfmt.patch.zip
git stash # Temporairly stash away any preexisting diff
git apply rustfmt.patch # Apply the diff
git add -u # Stage changed files
git commit -m "Applying rustfmt" # Commit the patch
git push
git stash pop # Unstast preexisting diff
rm rustfmt.patch.zip rustfmt.patch

Copy link

codecov bot commented Jun 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.69%. Comparing base (fb4e7e7) to head (5489cf2).
Report is 18 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
- Coverage   93.69%   93.69%   -0.01%     
==========================================
  Files          24       24              
  Lines        2809     2808       -1     
  Branches     2809     2808       -1     
==========================================
- Hits         2632     2631       -1     
  Misses          3        3              
  Partials      174      174              
Flag Coverage Δ
LLVM=16 93.63% <100.00%> (-0.01%) ⬇️
LLVM=19 93.69% <100.00%> (-0.01%) ⬇️
endianness=big 93.65% <100.00%> (-0.01%) ⬇️
endianness=little 93.63% <100.00%> (-0.01%) ⬇️
host=aarch64 93.63% <100.00%> (-0.01%) ⬇️
host=x86_64 93.69% <100.00%> (-0.01%) ⬇️
linux 93.69% <100.00%> (-0.01%) ⬇️
macos 93.63% <100.00%> (-0.01%) ⬇️
profile=dev 93.69% <100.00%> (-0.01%) ⬇️
profile=release 93.69% <100.00%> (-0.01%) ⬇️
rust=nightly 93.69% <100.00%> (-0.01%) ⬇️
rust=nightly-msvc 93.63% <100.00%> (-0.01%) ⬇️
rust=stable 93.69% <100.00%> (-0.01%) ⬇️
rust=stable-msvc 93.63% <100.00%> (-0.01%) ⬇️
target=aarch64 93.63% <100.00%> (-0.01%) ⬇️
target=powerpc64 93.65% <100.00%> (-0.01%) ⬇️
target=x86_64 93.63% <100.00%> (-0.01%) ⬇️
windows 93.63% <100.00%> (-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.

@LebedevRI LebedevRI merged commit b3e49ed into darktable-org:master Jun 27, 2025
92 of 93 checks passed
@LebedevRI LebedevRI deleted the develop branch June 27, 2025 23:15
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.

1 participant