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 branch lean-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