-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: fix semver checks in CI #1908
Conversation
ccfbe8d
to
79228c6
Compare
This PR contains breaking changes to the public Rust API. cargo-semver-checks summary
|
I think semver checks are working already? Note that workflow calls are executed in a separate runner, so installing things here won't change the runtime. |
Uhm. I see it failing here. https://github.com/CQCL/hugr/actions/runs/13155673562/job/36712137197#step:10:162 Do you know why it was working before? Is this a spurious error? |
I think it really fails earlier: I suspect that it has been broken since the github-runner-llvm issue |
Yes I agree this PR doesn't work. The best solution I can think of is to add |
See #1909 |
Alternative to #1908 I was a bit too eager when removing `capnp` in #1907. https://github.com/CQCL/hugr/pull/1907/files#diff-8c3de78a6f07b22f56457fe701ccc74e767b3e22be85276c500eb1a81c01281eL46-L47
No description provided.