Skip to content

update-subtree action: patch VeriFast proofs #371

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

Merged

Conversation

btj
Copy link

@btj btj commented May 29, 2025

When preparing a merge subtree PR, try to patch the VeriFast proofs.

Also tweaks patch-verifast-proofs.sh to ensure that no temporary files are left behind inside the working directory, even when the script fails mid-way.

Note: I did not test this.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

When preparing a merge subtree PR, try to patch the VeriFast proofs.

Note: I did not test this.
@btj btj requested a review from a team as a code owner May 29, 2025 20:25
@carolynzech carolynzech enabled auto-merge May 30, 2025 13:24
@carolynzech carolynzech added this pull request to the merge queue May 30, 2025
Merged via the queue into model-checking:main with commit 7139219 May 30, 2025
25 checks passed
@btj btj deleted the update-subtree-patch-verifast-proofs branch May 30, 2025 17:30
tautschnig added a commit to tautschnig/verify-rust-std that referenced this pull request Jun 2, 2025
PR model-checking#371 introduced invalid bash syntax ("line 51: syntax error near
unexpected token `fi'").
github-merge-queue bot pushed a commit that referenced this pull request Jun 2, 2025
PR #371 introduced invalid bash syntax ("line 51: syntax error near
unexpected token `fi'").

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 and MIT licenses.
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.

4 participants