Skip to content

Comments

Improve the CARGO_INCREMENTAL checking#2364

Merged
Xuanwo merged 2 commits intomozilla:mainfrom
anholt:increment
Mar 15, 2025
Merged

Improve the CARGO_INCREMENTAL checking#2364
Xuanwo merged 2 commits intomozilla:mainfrom
anholt:increment

Conversation

@anholt
Copy link
Contributor

@anholt anholt commented Mar 14, 2025

Improve the error message to give the user a better hint of what to do in response, and make it so you can sccache --help without throwing the error.

anholt added 2 commits March 14, 2025 11:10
It didn't give me much hint as to what was going wrong.
We should be able to sccache --help or sccache --show-stats even if
the env vars are set.
@codecov-commenter
Copy link

codecov-commenter commented Mar 14, 2025

Codecov Report

Attention: Patch coverage is 47.05882% with 9 lines in your changes missing coverage. Please review.

Project coverage is 67.60%. Comparing base (0cc0c62) to head (ce35504).
Report is 163 commits behind head on main.

Files with missing lines Patch % Lines
src/commands.rs 61.53% 5 Missing ⚠️
src/bin/sccache-dist/main.rs 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2364       +/-   ##
===========================================
+ Coverage   30.91%   67.60%   +36.68%     
===========================================
  Files          53       65       +12     
  Lines       20112    35471    +15359     
  Branches     9755        0     -9755     
===========================================
+ Hits         6217    23979    +17762     
- Misses       7922    11492     +3570     
+ Partials     5973        0     -5973     

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

Copy link
Collaborator

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, makes sense to me.

@Xuanwo Xuanwo merged commit 9fb3841 into mozilla:main Mar 15, 2025
60 checks passed
tottoto pushed a commit to tottoto/sccache that referenced this pull request Feb 6, 2026
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.

3 participants