Don't skip draft PR comment job for manual workflow trigger - #894
lewisgoddard merged 3 commits into
Conversation
Also find draft PR number for a manual workflow_dispatch when the pull_request event is not available.
|
Hit this issue (skipped comment step) when working on draft PR #873 but don't know how to test before merging, hopefully the change is ok? |
Installer builds for b8c1b1aDo not use these installers for production systems. They are intended for testing purposes only. They will update to the latest PR build, which will break your install eventually.
|
Installer builds for b8c1b1aDo not use these installers for production systems. They are intended for testing purposes only. They will update to the latest PR build, which will break your install eventually.
|
|
wooo @lewisgoddard looks like the workflow (commenting) succeeds for both ready and draft PRs as evidenced by the comments in this PR itself, the second one was run when in draft state and all comments' links are good. |
Installer builds for b8c1b1aDo not use these installers for production systems. They are intended for testing purposes only. They will update to the latest PR build, which will break your install eventually.
|
We don't actually need this, as it only runs if the builds completed.
Installer builds for 38060f8Do not use these installers for production systems. They are intended for testing purposes only. They will update to the latest PR build, which will break your install eventually.
|
2 participants
Also find draft PR number for a manual
workflow_dispatchwhen thepull_requestevent is not available.