Skip to content

fix: Unblock release-plz - #446

Open
day01 wants to merge 1 commit into
statrs-dev:mainfrom
day01:feat/fix-release-plz-lockfile
Open

fix: Unblock release-plz#446
day01 wants to merge 1 commit into
statrs-dev:mainfrom
day01:feat/fix-release-plz-lockfile

Conversation

@day01

@day01 day01 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Remove the tracked tests/no_std/Cargo.lock and stop using --locked for the no_std runtime build.

release-plz copies the repository using gitignore rules. Since *.lock is ignored, the tracked nested lockfile is omitted from the temporary copy and release-plz treats the repository as dirty.

The no_std runtime crate is only used for CI verification, so its dependencies can be resolved during the build.

@day01

day01 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

@YeungOnion post #403 regression

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.07%. Comparing base (92819b6) to head (82b8c41).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #446   +/-   ##
=======================================
  Coverage   95.07%   95.07%           
=======================================
  Files          62       62           
  Lines       14191    14191           
=======================================
  Hits        13492    13492           
  Misses        699      699           

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

@YeungOnion

Copy link
Copy Markdown
Contributor

I see how we can resolve deps onto a no_std target without locked, but how'd you catch the behavior with regards to release-plz? Part of the distributed package?

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