Closed dscho closed 1 year ago
The next steps will be to merge this (after dropping the TO-DROP
and Revert
commits that are at the top of the branch, to ease development by allowing to trigger the workflows even if they are in a non-default branch, for which you can trigger workflow_dispatch
only if the default branch has a workflow of the same name), and then to adjust https://github.com/git-for-windows/gfw-helper-github-app/'s /git-artifacts
and /release
commands to use the new workflows.
@rimrul FWIW my plan is to merge this PR today (after squashing the fixup
s/amend
s), and probably also https://github.com/git-for-windows/gfw-helper-github-app/pull/29.
This has been a huge lift for me, therefore I might not actually be up to going through with the /git-artifacts
//release
invocations today, in particular given that the Git maintainer is in a timezone that's 9h behind mine.
I'll try to get a review for https://github.com/git-for-windows/gfw-helper-github-app/pull/29. and a final review for this PR in. But the latter will likely be close to the scheduled release time for 2.40.0-rc2.
I'll try to get a review for git-for-windows/gfw-helper-github-app#29. and a final review for this PR in. But the latter will likely be close to the scheduled release time for 2.40.0-rc2.
@rimrul thank you so much!
Would you be okay with me merging the PRs when I need them, and implementing any changes in response to your review in follow-up PRs?
Yes.
Okay, let's go.
Full disclosure: these are the fixes that were needed after merging this PR: https://github.com/git-for-windows/git-for-windows-automation/compare/69b0b5f47de9ba1b9bfbabf3df496487c22be181...f2ef37027006581c06999eeed3c7fad9120f66cd
tag-git
workflow so that it can tag full Git for Windows versions, not just snapshotsgit-artifacts
workflow to mirror Check Runs togit-for-windows/git
(needs to encode the architecture as part of the Check Run name to avoid them overwriting each other)release-git
This addresses https://github.com/git-for-windows/git-for-windows-automation/issues/37