Zulip Chat Archive
Stream: nightly-testing
Topic: what to do?
Jon Eugster (Aug 26 2024 at 15:39):
I've tried to read the documentation about lean-pr-testing-...
branches, and I'm massively confused. Could I please get some instructions on whether I'm doing this right?
So there is Lean PR lean4#5173 and I've pushed a commit with fixes to the mathlib branch lean-pr-testing-5173
.. and now what?
Is it correct that I've created mathlib PR #16157 which goes bump/v4.12.0 <-- lean-pr-testing-5173
?
Ruben Van de Velde (Aug 26 2024 at 15:45):
I think we need to merge lean-pr-testing-#### into nightly-testing first
Ruben Van de Velde (Aug 26 2024 at 15:46):
I think you can change the target branch of your pr
Jon Eugster (Aug 26 2024 at 15:52):
ok. and do I do that now or after the PR is merged in core?
What confused me is that it says nightly-testing <-- lean-pr-testing-5173
"can be done anytime", but I guess I'll just make the PR and then wait for somebody of yous to use that when the time is right.
Ruben Van de Velde (Aug 26 2024 at 18:46):
Yeah, make the PR to nightly-testing, and then when the nightly with your changes is released, a bot will bump the version on the nightly-testing branch, which will fail - at that point, we can merge your PR
Kim Morrison (Aug 26 2024 at 22:07):
The essential step for contributors is to get a builds-mathlib
label on their lean PR.
Kim Morrison (Aug 26 2024 at 22:11):
The steps for that are:
- Make sure your lean PR is branched of
nightly-with-mathlib
(rebase it if not). - If you get a
breaks-mathlib
, then you'll need to push changes to the Mathlib branchlean-pr-testing-NNNN
branch until it's fixed - If the breakage is in Batteries, then PR to that same name branch there (and ping me so I can merge quickly), but then also on the Mathlib branch update the lakefiles to point Batteries at that branch, and run
lake update batteries
Last updated: May 02 2025 at 03:31 UTC