Zulip Chat Archive

Stream: lean4 dev

Topic: FYI CI now merges master


Gabriel Ebner (Feb 24 2023 at 03:11):

I didn't realize that until now but apparently the upgrade to actions/checkout@v3 changed its default behavior.

Gabriel Ebner (Feb 24 2023 at 03:12):

It now merges master into the PR commit.

Sebastian Ullrich (Feb 24 2023 at 08:27):

I'm quite sure it already did that before

Gabriel Ebner (Feb 24 2023 at 18:24):

It certainly doesn't happen in mathlib though. Maybe it's because we're using the auto merge feature in lean4?

Sebastian Ullrich (Feb 25 2023 at 09:27):

Mmh interesting, the instructions on how to do that explicitly seem to be unchanged from v2 to v3 https://github.com/actions/checkout/tree/v2.6.0#Checkout-pull-request-HEAD-commit-instead-of-merge-commit


Last updated: Dec 20 2023 at 11:08 UTC