Skip to content

Adding an overall timeout option#483

Merged
marcoeilers merged 3 commits into
masterfrom
meilers_timeout
Dec 7, 2023
Merged

Adding an overall timeout option#483
marcoeilers merged 3 commits into
masterfrom
meilers_timeout

Conversation

@marcoeilers

Copy link
Copy Markdown
Contributor

Adding a command line option to give an overall timeout for the Boogie verification.
Previously, Carbon had no such option itself, and Boogie only offers a timeout per procedure.

@marcoeilers

Copy link
Copy Markdown
Contributor Author

It's probably nicer if run returns an option or an Either to represent timeouts instead of a specific string; I'll change that tomorrow.

@gauravpartha gauravpartha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.

@marcoeilers marcoeilers merged commit 6e4ce2f into master Dec 7, 2023
@marcoeilers marcoeilers deleted the meilers_timeout branch December 7, 2023 16:50
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