From e56525199190bcbb1498df07d3fa4fcb3fdff831 Mon Sep 17 00:00:00 2001 From: Ludy Date: Sat, 1 Aug 2026 15:28:47 +0200 Subject: [PATCH] ci(tauri): disable updater signing for unsigned builds (#7181) # Description of Changes This change updates the Tauri CI workflow to: - Disable Tauri updater artifact generation for unsigned builds. - Prevent unsigned builds from requiring updater signing secrets. - Skip PR artifact comments for fork and Dependabot pull requests, where the GitHub token is read-only. - Preserve signed release builds and their updater artifacts unchanged. The change prevents unsigned CI builds from failing because of unavailable or invalid updater signing credentials. It also prevents expected GitHub API permission errors from failing the workflow. --- ## 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/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 run `task check` to verify linters, typechecks, and tests pass - [ ] I have tested my changes locally. Refer to the [Testing Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/DeveloperGuide.md#7-testing) for more details. --- .github/workflows/tauri-build.yml | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/.github/workflows/tauri-build.yml b/.github/workflows/tauri-build.yml index 7daaf18bc1..f7763b3dbe 100644 --- a/.github/workflows/tauri-build.yml +++ b/.github/workflows/tauri-build.yml @@ -420,7 +420,9 @@ jobs: # Linux: build deb+rpm only here (deb-only on minimal smoke builds). # AppImage runs in its own continue-on-error step below so its # persistent linuxdeploy failure (#6127 onwards) does not tank uploads. - args: ${{ matrix.platform == 'ubuntu-22.04' && (inputs.minimal && '--bundles deb' || '--bundles deb,rpm') || matrix.args }} + args: >- + ${{ matrix.platform == 'ubuntu-22.04' && (inputs.minimal && '--bundles deb' || '--bundles deb,rpm') || matrix.args }} + --config '{"bundle":{"createUpdaterArtifacts":false}}' # AppImage is decoupled so its linuxdeploy run gets a fresh process # (rpm scratch state torn down) and its failure can't tank deb/rpm. @@ -613,7 +615,14 @@ jobs: pr-comment: needs: build runs-on: ubuntu-latest - if: github.event_name == 'pull_request' && needs.build.result == 'success' + # Fork and Dependabot pull_request runs receive a read-only GITHUB_TOKEN, + # so the API cannot create or update PR comments there. The artifacts are + # still uploaded and remain available from the Actions run page. + if: >- + github.event_name == 'pull_request' && + needs.build.result == 'success' && + !github.event.pull_request.head.repo.fork && + github.actor != 'dependabot[bot]' permissions: pull-requests: write steps: