Skip to content

Make Cargo-minimal and Cargo-recent lockfiles follow expected convention#1612

Open
benalleng wants to merge 6 commits into
payjoin:masterfrom
benalleng:lockfile-correctness
Open

Make Cargo-minimal and Cargo-recent lockfiles follow expected convention#1612
benalleng wants to merge 6 commits into
payjoin:masterfrom
benalleng:lockfile-correctness

Conversation

@benalleng

@benalleng benalleng commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

The Cargo-minimal and Cargo-recent lockfiles were not actually pointing to the minimal and recent semvers of the dependencies and were instead just copies of the Cargo.lock lockfile.

This is somewhat related to the release but I don't think its all that critical just something that I thought should be improved.

Relates to #940 though I'm not sure it should close it
also related is #454

Pull Request Checklist

Please confirm the following before requesting review:

@coveralls

coveralls commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 27285085924

Coverage remained the same at 85.244%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 14699
Covered Lines: 12530
Line Coverage: 85.24%
Coverage Strength: 371.43 hits per line

💛 - Coveralls

@benalleng benalleng force-pushed the lockfile-correctness branch from 13a6c1c to 31dd86b Compare June 6, 2026 18:45
@benalleng benalleng marked this pull request as draft June 6, 2026 19:04
@benalleng benalleng force-pushed the lockfile-correctness branch 4 times, most recently from 0eff67a to e0182f0 Compare June 8, 2026 18:12
@benalleng benalleng force-pushed the lockfile-correctness branch 6 times, most recently from 1735564 to 36266b3 Compare June 10, 2026 14:06
@benalleng benalleng force-pushed the lockfile-correctness branch from 36266b3 to 3e77c6c Compare June 10, 2026 14:53
@benalleng benalleng marked this pull request as ready for review June 10, 2026 15:02
@benalleng benalleng requested a review from DanGould June 10, 2026 15:03
@benalleng

Copy link
Copy Markdown
Collaborator Author

This should avoid cargo updates from occuring on stray CI jobs preventing failures like https://github.com/payjoin/rust-payjoin/actions/runs/27285126244/job/80590095693?pr=1629 in efc5b92 from happening

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