Skip to content

make rectify add checksums when they are not there yet - #406

Merged
nevrome merged 2 commits into
rectifyModifySplitfrom
rectifyAddChecksums
Sep 29, 2026
Merged

nevrome merged 2 commits into
rectifyModifySplitfrom
rectifyAddChecksums

Conversation

@nevrome

@nevrome nevrome commented Sep 26, 2026

Copy link
Copy Markdown
Member

No description provided.

@nevrome
nevrome added this pull request to stack #407 September 26, 2026 16:56
@nevrome
nevrome requested a review from stschiff September 26, 2026 16:56
@nevrome

nevrome commented Sep 26, 2026

Copy link
Copy Markdown
Member Author

Continuing the discussion from #401 I now think that rectify should indeed just add missing checksums. It's easier, overall.

@codecov

codecov Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 52.53%. Comparing base (8e50a54) to head (f58aa7d).

Files with missing lines Patch % Lines
src/Poseidon/Core/Package.hs 83.33% 1 Missing ⚠️
Additional details and impacted files
@@                  Coverage Diff                   @@
##           rectifyModifySplit     #406      +/-   ##
======================================================
+ Coverage               52.48%   52.53%   +0.04%     
======================================================
  Files                      38       38              
  Lines                    6127     6129       +2     
  Branches                  655      654       -1     
======================================================
+ Hits                     3216     3220       +4     
+ Misses                   2256     2255       -1     
+ Partials                  655      654       -1     

☔ View full report in Codecov by Harness.
📢 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.

@stschiff stschiff left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! Please merge, I'll review all of this again in rectifyModifySplit before merging into master.

@nevrome
nevrome force-pushed the rectifyAddChecksums branch from a2a11da to f58aa7d Compare September 29, 2026 09:10
@nevrome
nevrome removed this pull request from stack #407 September 29, 2026 11:45
@nevrome
nevrome merged commit 8a9e442 into rectifyModifySplit Sep 29, 2026
4 checks passed
@nevrome
nevrome deleted the rectifyAddChecksums branch September 29, 2026 11:46
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