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