From cfe90cff66bf12abc8e93fadb6b91cf9ca4cd148 Mon Sep 17 00:00:00 2001 From: Ludy Date: Tue, 13 Jan 2026 17:58:50 +0100 Subject: [PATCH] fix(ci): read workflow_dispatch PR number from inputs.pr in PR auto-deploy workflow (#5380) # Description of Changes This pull request makes a small update to the PR auto-deploy workflow. The change ensures that the pull request number is correctly retrieved from the workflow dispatch payload, improving reliability when the workflow is manually triggered. * Changed the PR number retrieval logic in `.github/workflows/PR-Auto-Deploy-V2.yml` to use `context.payload.inputs.pr` instead of `process.env.INPUT_PR` for workflow dispatch events. --- ## Checklist ### General - [ ] I have read the [Contribution Guidelines](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/CONTRIBUTING.md) - [ ] I have read the [Stirling-PDF Developer Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/DeveloperGuide.md) (if applicable) - [ ] I have read the [How to add new languages to Stirling-PDF](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/HowToAddNewLanguage.md) (if applicable) - [ ] I have performed a self-review of my own code - [ ] My changes generate no new warnings ### Documentation - [ ] I have updated relevant docs on [Stirling-PDF's doc repo](https://github.com/Stirling-Tools/Stirling-Tools.github.io/blob/main/docs/) (if functionality has heavily changed) - [ ] I have read the section [Add New Translation Tags](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/HowToAddNewLanguage.md#add-new-translation-tags) (for new translation tags only) ### Translations (if applicable) - [ ] I ran [`scripts/counter_translation.py`](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/docs/counter_translation.md) ### UI Changes (if applicable) - [ ] Screenshots or videos demonstrating the UI changes are attached (e.g., as comments or direct attachments in the PR) ### Testing (if applicable) - [ ] I have tested my changes locally. Refer to the [Testing Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/devGuide/DeveloperGuide.md#6-testing) for more details. --- .github/config/repo_devs.json | 4 +++- .github/workflows/PR-Auto-Deploy-V2.yml | 6 +++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/.github/config/repo_devs.json b/.github/config/repo_devs.json index 963240260..8b0bb97a8 100644 --- a/.github/config/repo_devs.json +++ b/.github/config/repo_devs.json @@ -13,6 +13,8 @@ "reecebrowne", "DarioGii", "ConnorYoh", - "EthanHealy01" + "EthanHealy01", + "jbrunton96", + "balazs-szucs" ] } diff --git a/.github/workflows/PR-Auto-Deploy-V2.yml b/.github/workflows/PR-Auto-Deploy-V2.yml index 8d9f837f6..9f30b372f 100644 --- a/.github/workflows/PR-Auto-Deploy-V2.yml +++ b/.github/workflows/PR-Auto-Deploy-V2.yml @@ -11,6 +11,10 @@ on: allow_fork: description: "Allow deploying fork PR?" required: false + type: choice + options: + - "true" + - "false" default: "false" permissions: @@ -42,7 +46,7 @@ jobs: script: | const { owner, repo } = context.repo; let prNumber = context.eventName === 'workflow_dispatch' - ? parseInt(process.env.INPUT_PR, 10) + ? parseInt(context.payload.inputs.pr, 10) : context.payload.number; if (!Number.isInteger(prNumber)) { core.setFailed('Invalid PR number'); return; }