Skip to content
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

CI: Install cbindgen #339

Merged

Conversation

chrysn
Copy link
Collaborator

@chrysn chrysn commented Feb 2, 2025

On top of #337, this installs cbindgen, which is apparently still missing.

@chrysn
Copy link
Collaborator Author

chrysn commented Feb 2, 2025

@malishav, can you merge malishav/rust-psa-crypto#1? Then I can rebase that and drop the #337 fix, and we got CI back.

@chrysn chrysn changed the title Install cbindgen CI: Install cbindgen Feb 2, 2025
@malishav
Copy link
Contributor

malishav commented Feb 2, 2025

Merged!

@chrysn
Copy link
Collaborator Author

chrysn commented Feb 2, 2025

Removed the commit that switches to my branch.

I don't know how anything would go wrong now, but still, let's see what CI says. If all is green, we have unblocked the other issues; I think we'll have to rebase all pending PRs once this is merged.

@malishav malishav merged commit 8949331 into openwsn-berkeley:main Feb 2, 2025
37 checks passed
@chrysn chrysn deleted the install-cbindgen branch February 3, 2025 05:49
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