chore: Update workflows to current version in template repository#1097
chore: Update workflows to current version in template repository#1097DerDreschner merged 1 commit intomasterfrom
Conversation
Signed-off-by: David Dreschner <david.dreschner@nextcloud.com>
There was a problem hiding this comment.
Pull request overview
This PR syncs this repository’s GitHub Actions workflows with the current versions used in the Nextcloud template repository, primarily by bumping pinned action SHAs (and a few workflow parameters) to their newer equivalents.
Changes:
- Bump pinned GitHub Actions dependencies across workflows (e.g.,
actions/checkout,shivammathur/setup-php,dorny/paths-filter,actions/setup-node,peter-evans/create-pull-request). - Update
lint-php.ymlto derive a min/max PHP version and run linting against those two versions. - Update
npm-audit-fix.ymlbranch matrix and Node/npm fallback versions.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/reuse.yml | Bumps actions/checkout pin used for REUSE compliance job. |
| .github/workflows/psalm.yml | Updates checkout, PHP setup, and version-matrix action pins for static analysis. |
| .github/workflows/phpunit-sqlite.yml | Updates checkout/setup-php/version-matrix and paths-filter pins used in SQLite CI. |
| .github/workflows/phpunit-pgsql.yml | Updates checkout/setup-php/version-matrix and paths-filter pins used in PostgreSQL CI. |
| .github/workflows/phpunit-mariadb.yml | Updates checkout/setup-php/version-matrix and paths-filter pins used in MariaDB CI. |
| .github/workflows/npm-audit-fix.yml | Updates checkout/setup-node/create-pull-request pins, changes fallback Node/npm, and adjusts branch matrix. |
| .github/workflows/lint-php.yml | Updates action pins and changes the PHP matrix strategy to min/max versions. |
| .github/workflows/lint-php-cs.yml | Updates checkout/setup-php/version-matrix pins for PHP-CS linting. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -28,14 +28,13 @@ jobs: | |||
| - ${{ github.event.repository.default_branch }} | |||
| - 'stable33' | |||
| - 'stable32' | |||
There was a problem hiding this comment.
The workflow matrix no longer includes the stable31 branch, but this repo still references stable31 as a supported/base branch (e.g., in renovate.json). If stable31 is still maintained, it will silently stop receiving the scheduled npm-audit-fix updates. Consider adding stable31 back to the matrix or, if stable31 is truly EOL, update the other branch lists/configs accordingly to keep them consistent.
| - 'stable32' | |
| - 'stable32' | |
| - 'stable31' |
There was a problem hiding this comment.
I think we should modify the renovate.json as Nextcloud 31 is EOL?
| @@ -28,14 +28,13 @@ jobs: | |||
| - ${{ github.event.repository.default_branch }} | |||
| - 'stable33' | |||
| - 'stable32' | |||
This pull request updates all workflows with the current version in the template repository. I'll hope the failing
static-psalm-analysisworkflow will work again afterwards.