8000 Run kani daily on a schedule by tcharding · Pull Request #1511 · rust-bitcoin/rust-bitcoin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Run kani daily on a schedule #1511

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
merged 2 commits into from
Dec 30, 2022
Merged

Conversation

tcharding
Copy link
Member

Running kani takes ages, instead of running it on every pull request we can just run it daily.

Running kani takes ages, instead of running it on every pull request we
can just run it daily.
@tcharding
Copy link
Member Author

I don't actually know how to view the results of a the last scheduled run?

@sanket1729
Copy link
Member
sanket1729 commented Dec 29, 2022

I presume it should show up in the actions tab? It might require us to manually check the results

In order for us to easily see the result of last nights kani run add a
badge to the readme file.
@tcharding
Copy link
Member Author
tcharding commented Dec 29, 2022

Thanks @sanket1729, that gave me the idea of adding a badge to the readme. I've not done badges before so I just guessed based on: https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/adding-a-workflow-status-badge

Copy link
Member
@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

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

ACK 3372333. Even if the badge is not displayed correctly, we can debug in follow up PRs.

It is possible to double-check it with a fork by replacing
https://github.com/$user/rust-bitcoin/actions/workflows/kani.yaml/badge.svg, but looks correct to me.

@apoelstra
Copy link
Member

It's possible to get github notifications for all CI runs, which I've turned on for this repo. Presumably I'll see these scheduled runs.

The notifications are a little annoying since you get several of them every time a PR is merged ... but on the other hand they're easy to click away and it's nice to see the notification list rapidly shrink. There's probably a "only notify on failure" option.

Copy link
Member
@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 3372333

Copy link
Member
@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

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

ACK 3372333

@apoelstra apoelstra merged commit 86eda8f into rust-bitcoin:master Dec 30, 2022
@tcharding tcharding deleted the 12-29-cron-kani branch January 12, 2023 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0