Skip to content

Commit cb86e94

Browse files
committed
fix: mark as main PR needed when target quarkus platform is main
Signed-off-by: Chris Laprun <[email protected]>
1 parent 0d41c68 commit cb86e94

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.github/workflows/release-update-quarkus-platform.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
qosdk-bot-token: ${{ secrets.qosdk-bot-token }}
2727

2828
update-quarkus-platform-main-if-needed:
29-
if: ${{ inputs.needs-main-pr }}
29+
if: ${{ inputs.needs-main-pr && inputs.quarkus-platform-branch != 'main' }}
3030
uses: ./.github/workflows/release-quarkus-platform-pr.yml
3131
with:
3232
qosdk-version: ${{inputs.qosdk-version}}

.github/workflows/release.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,12 @@ jobs:
7373
7474
if [ "${quarkus_platform_branch}" = "main" ]; then
7575
echo "quarkus-platform update will target main"
76+
echo "need_main_pr=true" >> $GITHUB_OUTPUT
7677
else
7778
echo "quarkus-platform update will target ${quarkus_platform_branch}"
7879
echo "Check if quarkus-platform also needs to be updated on main"
7980
if [ "${{ github.base_ref }}" = "main" ]; then
80-
echo "PR targes main so assume quarkus-platform main also needs to be updated"
81+
echo "PR targets main so assume quarkus-platform main also needs to be updated"
8182
echo "need_main_pr=true" >> $GITHUB_OUTPUT
8283
else
8384
echo "No need for PR on quarkus-platform main"

0 commit comments

Comments
 (0)