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][kani] Use kani-github-action #472

Merged
merged 7 commits into from
Oct 12, 2023
Merged

[ci][kani] Use kani-github-action #472

merged 7 commits into from
Oct 12, 2023

Conversation

Agnik7
Copy link
Contributor

@Agnik7 Agnik7 commented Oct 6, 2023

Removes the hand-rolled Kani CI job and replaces it with the official kani-github-action.

Resolves #423

@Agnik7
Copy link
Contributor Author

Agnik7 commented Oct 6, 2023

@joshlf requesting your review in this PR. Apologies for any inconvenience caused

Copy link
Member

@joshlf joshlf left a comment

Choose a reason for hiding this comment

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

Thanks for putting this up! I think we can simplify this some, but this is definitely on the right track.

@Agnik7
Copy link
Contributor Author

Agnik7 commented Oct 7, 2023

@joshlf I have made the changes. Kindly review.

@Agnik7 Agnik7 requested a review from joshlf October 8, 2023 03:58
@Agnik7
Copy link
Contributor Author

Agnik7 commented Oct 8, 2023

@joshlf requesting for a follow up. Apologies for any inconvenience caused.

@Agnik7
Copy link
Contributor Author

Agnik7 commented Oct 9, 2023

@joshlf I have made the changes. Kindly review

@joshlf
Copy link
Member

joshlf commented Oct 9, 2023

Okay everything looks good now; not sure why CI is still failing. Maybe try pulling the latest changes from main and rebasing on them?

@Agnik7
Copy link
Contributor Author

Agnik7 commented Oct 10, 2023

@joshlf I have already rebased the branch, so there should be no conflict.

@joshlf
Copy link
Member

joshlf commented Oct 11, 2023

@joshlf I have already rebased the branch, so there should be no conflict.

Hmm interesting. In that case, I have no idea why this job is failing. I'm going to try rebasing through the UI again to see if that magically fixes things 🤷‍♂️

@joshlf joshlf changed the title fix: Update workflow to use kani-github-action in CI [ci][kani] Use kani-github-action Oct 12, 2023
@Agnik7
Copy link
Contributor Author

Agnik7 commented Oct 12, 2023

@joshlf the jobs have passed!! Thanks a lot for your help and guidance

@joshlf joshlf enabled auto-merge October 12, 2023 00:18
Copy link
Member

@joshlf joshlf left a comment

Choose a reason for hiding this comment

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

Okay, looks like it was a bug on main that your PR just exposed. I've fixed the bug and rebased this PR on that fix, and now everything works!

@joshlf joshlf added this pull request to the merge queue Oct 12, 2023
Merged via the queue into google:main with commit e87fa92 Oct 12, 2023
@joshlf
Copy link
Member

joshlf commented Oct 12, 2023

Thanks again, @Agnik7!

@Agnik7
Copy link
Contributor Author

Agnik7 commented Oct 12, 2023

Thanks a lot @joshlf for the opportunity and the guidance you provided!!

samuelselleck pushed a commit to samuelselleck/zerocopy that referenced this pull request Oct 13, 2023
* Update ci.yml

* Update ci.yml

* Update ci.yml

---------

Co-authored-by: Joshua Liebow-Feeser <[email protected]>
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.

Use kani-github-action for Kani in CI.
2 participants