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

Build patterns nightly to get alert if it fails #70

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

SeanTAllen
Copy link
Member

Currently, a person opening a PR can be surprised by failures unrelated to their changes. In particular, to URLs that are external to the site no longer working and failing validation.

Worse, this means that readers of pony patterns are experiencing broken URLs that won't be discovered until someone does an update for the patterns and the build fails.

This commit adds a nightly job that tries to build the site and if it fails, alerts to Zulip so we can be made aware of issues in a timely fashion and can endeavor to fix them as soon as possible.

The workflow can also be triggered manually if need be.

Currently, a person opening a PR can be surprised by failures unrelated
to their changes. In particular, to URLs that are external to the site
no longer working and failing validation.

Worse, this means that readers of pony patterns are experiencing broken
URLs that won't be discovered until someone does an update for the
patterns and the build fails.

This commit adds a nightly job that tries to build the site and if it
fails, alerts to Zulip so we can be made aware of issues in a timely
fashion and can endeavor to fix them as soon as possible.

The workflow can also be triggered manually if need be.
@SeanTAllen SeanTAllen requested a review from a team December 10, 2023 05:22
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Dec 10, 2023
@SeanTAllen SeanTAllen merged commit 06703dc into main Dec 12, 2023
10 checks passed
@SeanTAllen SeanTAllen deleted the build-nightly-to-find-busted branch December 12, 2023 19:07
@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Dec 12, 2023
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