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

Add CBMC proof-running GitHub Action #137

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

karkhaz
Copy link
Contributor

@karkhaz karkhaz commented Feb 17, 2023

This commit adds a GitHub Action that runs the CBMC proofs upon every push and pull request. This is intended to replace the current CBMC CI.

@karkhaz karkhaz force-pushed the kk-add-github-action branch 9 times, most recently from 5e6a762 to f009a08 Compare March 1, 2023 20:29
@karkhaz karkhaz marked this pull request as ready for review March 1, 2023 21:10
@karkhaz karkhaz requested a review from a team as a code owner March 1, 2023 21:10
@karkhaz karkhaz force-pushed the kk-add-github-action branch from f009a08 to 4afcd3a Compare March 3, 2023 20:23
if proc.returncode:
logging.critical("Could not add tool version printing job")
sys.exit(1)
#def add_tool_version_job():
Copy link
Member

Choose a reason for hiding this comment

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

Why is this code commented out? Should it be removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi Archit, I'm planning on re-enabling this for all repositories (not all of them currently have it), but in a separate PR. Is that OK?

Copy link
Member

Choose a reason for hiding this comment

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

That is fine; in that case would be best for this PR to remove the commented out code, and can add it back in the other PR when ready

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, done. Thank you

This commit adds a GitHub Action that runs the CBMC proofs in this
repository upon pushes and pull requests
@karkhaz karkhaz force-pushed the kk-add-github-action branch from 4afcd3a to 0986ef6 Compare March 3, 2023 22:38
@karkhaz karkhaz merged commit 360cfcd into FreeRTOS:main Mar 3, 2023
@karkhaz karkhaz deleted the kk-add-github-action branch March 3, 2023 23:38
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.

3 participants