Skip to content

Conversation

@SirLynix
Copy link
Contributor

@SirLynix SirLynix commented Sep 9, 2025

Implement --skip-unchanged from #74

@codecov
Copy link

codecov bot commented Sep 9, 2025

Codecov Report

❌ Patch coverage is 88.67925% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.91%. Comparing base (ea6e9a6) to head (c928a9a).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/ShaderArchiver/Archiver.cpp 87.50% 0 Missing and 3 partials ⚠️
src/ShaderCompiler/Compiler.cpp 86.95% 0 Missing and 3 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
- Coverage   81.13%   76.91%   -4.23%     
==========================================
  Files         146      200      +54     
  Lines       18337    24895    +6558     
  Branches        0     7490    +7490     
==========================================
+ Hits        14878    19148    +4270     
- Misses       3459     5631    +2172     
- Partials        0      116     +116     
Files with missing lines Coverage Δ
src/NZSL/Parser.cpp 88.76% <100.00%> (+0.90%) ⬆️
src/ShaderArchiver/Archiver.hpp 100.00% <ø> (ø)
src/ShaderCompiler/Compiler.hpp 100.00% <ø> (ø)
tests/src/Tests/NzslaTests.cpp 100.00% <100.00%> (ø)
tests/src/Tests/NzslcTests.cpp 96.77% <100.00%> (ø)
src/ShaderArchiver/Archiver.cpp 70.77% <87.50%> (ø)
src/ShaderCompiler/Compiler.cpp 53.42% <86.95%> (ø)

... and 142 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ea6e9a6...c928a9a. Read the comment docs.

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

@SirLynix SirLynix force-pushed the tools-skip-unchanged branch from b8acf0c to bc27232 Compare September 10, 2025 06:08
Introduces a --skip-unchanged option to nzsla and nzslc, preventing overwriting output files if the content has not changed.
@SirLynix SirLynix force-pushed the tools-skip-unchanged branch from bc27232 to c928a9a Compare September 10, 2025 06:11
@SirLynix SirLynix merged commit 04b584b into main Sep 10, 2025
129 of 132 checks passed
@SirLynix SirLynix deleted the tools-skip-unchanged branch September 10, 2025 09:43
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