Skip to content

top: fix warning from manual_saturating_arithmetic#647

Merged
cakebaker merged 1 commit intouutils:mainfrom
cakebaker:top_fix_manual_saturating_arithmetic
Mar 10, 2026
Merged

top: fix warning from manual_saturating_arithmetic#647
cakebaker merged 1 commit intouutils:mainfrom
cakebaker:top_fix_manual_saturating_arithmetic

Conversation

@cakebaker
Copy link
Contributor

This PR fixes a warning from the manual_saturating_arithmetic lint

@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 18.78%. Comparing base (6ac0e01) to head (60a0f1d).
⚠️ Report is 15 commits behind head on main.

Files with missing lines Patch % Lines
src/uu/top/src/tui/mod.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #647      +/-   ##
==========================================
+ Coverage   18.77%   18.78%   +0.01%     
==========================================
  Files          66       66              
  Lines        9752     9745       -7     
  Branches      516      517       +1     
==========================================
  Hits         1831     1831              
+ Misses       7921     7914       -7     
Flag Coverage Δ
macos_latest 6.91% <0.00%> (ø)
ubuntu_latest 19.15% <0.00%> (+0.01%) ⬆️
windows_latest 0.02% <0.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.

@cakebaker cakebaker merged commit 2cd7998 into uutils:main Mar 10, 2026
17 of 18 checks passed
@cakebaker cakebaker deleted the top_fix_manual_saturating_arithmetic branch March 10, 2026 09:01
@cakebaker
Copy link
Contributor Author

I merged this PR myself to get the CI back to green.

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