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

[Merged by Bors] - ci: automated nightly-testing branch #6823

Closed
wants to merge 5 commits into from

Conversation

semorrison
Copy link
Contributor

@semorrison semorrison commented Aug 28, 2023

This PR adds three workflows that manage a nightly-testing branch.

  • Every commit to master is merged to nightly-testing, resolving conflicts in favour of nightly-testing.
  • Once a day the lean-toolchain is bumped to the newest Lean nightly release.
  • If CI fails, a bot posts in the mathlib reviewers stream.

The aim is to get quick notification that a nightly release of Lean is incompatible with Mathlib.

The nightly-testing branch is not intended to ever contain human contributions, and in particular it is always okay to delete it and recreate it from master.

If there are breaking changes:

  • If it could be fixed by a change to Mathlib that would work both on the current lean-toolchain used by master, and the latest nightlies, please make the PR to Mathlib master.
  • If it requires a fix that doesn't make sense on the current lean-toolchain used by master, please make a PR with a new lean-toolchain, mark that PR as blocked-by-core-release, but also merge your PR into nightly-testing (once you've verified it works!)

There is no attempt to avoid repeated messages if we've already notified zulip about the failure, and subsequent commits continue to fail. Suggestions for dealing with that welcome.


Open in Gitpod

@semorrison semorrison added awaiting-review RFC Request for comment CI Modifies the continuous integration / deployment setup and removed RFC Request for comment labels Aug 28, 2023
@jcommelin
Copy link
Member

There is no attempt to avoid repeated messages if we've already notified zulip about the failure, and subsequent commits continue to fail. Suggestions for dealing with that welcome.

Since the CI runs only once a day, I think a daily nag is actually quite good. So I think the repeated messages are a feature, not a bug.

@jcommelin
Copy link
Member

Thanks 🎉

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Sep 1, 2023
bors bot pushed a commit that referenced this pull request Sep 1, 2023
This PR adds three workflows that manage a `nightly-testing` branch.

* Every commit to `master` is merged to `nightly-testing`, resolving conflicts in favour of `nightly-testing`.
* Once a day the `lean-toolchain` is bumped to the newest Lean nightly release.
* If CI fails, a bot posts in the mathlib reviewers stream.

The aim is to get quick notification that a nightly release of Lean is incompatible with Mathlib.

The `nightly-testing` branch is not intended to ever contain human contributions, and in particular it is always okay to delete it and recreate it from `master`.

If there are breaking changes:
* If it could be fixed by a change to Mathlib that would work both on the current `lean-toolchain` used by `master`, and the latest nightlies, please make the PR to Mathlib `master`.
* If it requires a fix that doesn't make sense on the current `lean-toolchain` used by `master`, please make a PR with a new `lean-toolchain`, mark that PR as `blocked-by-core-release`, but also **merge** your PR into `nightly-testing` (once you've verified it works!)

There is no attempt to avoid repeated messages if we've already notified zulip about the failure, and subsequent commits continue to fail. Suggestions for dealing with that welcome.



Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@bors
Copy link

bors bot commented Sep 1, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title ci: automated nightly-testing branch [Merged by Bors] - ci: automated nightly-testing branch Sep 1, 2023
@bors bors bot closed this Sep 1, 2023
@bors bors bot deleted the nightly_workflows branch September 1, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Modifies the continuous integration / deployment setup ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants