Repository navigation
ci: centralize upstream PHPStan, PHPUnit and Playwright templates - #115
Merged
Merged
Conversation
Avoid replacing catalog-locked workflow files or advancing the lock for a failed patch; propagate blocked status to the synchronizer. Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
…ht templates Track immutable Nextcloud sources with checksums; render public runners in the LibreCode workflow catalog and distribute via the existing template synchronizer. Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
vitormattos
added a commit
to LibreCodeCoop/communications
that referenced
this pull request
Nov 15, 2026
…ibreCode catalog Reduce six application patches to a single Playwright fixture overlay, use the shared catalog lock and preserve upstream PHPStan/Composer contracts. Depends on LibreCodeCoop/.github#115. Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Centralize the upstream-derived GitHub Actions workflows for PHPStan, PHPUnit (SQLite/MariaDB/PostgreSQL) and Playwright in LibreCodeCoop/.github. Consumers should not maintain copies of runner changes or duplicate the official Nextcloud templates. This changes only LibreCode-owned files; nothing is committed or opened in nextcloud/.github.
Changes
c2798fc61844a6420779add9f5e1b51f8aabfd87, with SHA-256 pins inupstream/sources.json.upstream/templates.json. A single generic runner alias mapping normalizesruns-on: ubuntu-latest-lowto GitHub-hostedubuntu-latestafter patch processing, leaving the original vendored file and other text unchanged. No new local PHP/Playwright patches are required for this runner change.actions/sync-workflows/sync.py: it now renders consumer patches in isolation and never overwrites the last valid workflow or updates its lock if patch application fails; failures block synchronization instead of producing potentially broken CI.Validation
python3 -m unittest discover -s tests -p 'test_*.py' -q: 109 tests passed.python3 scripts/sync_upstream.py check upstream/sources.json: passed; vendored sources match recorded immutable SHA-256 digests.python3 scripts/render_upstream.py check upstream/templates.json: passed.actionlintfor all five new generated workflow files: passed.python3 scripts/check_workflow_policy.py: passed.Consumer migration
Once merged,
LibreCodeCoop/communications#1can switch back to the standard organization sync workflow, remove its extra Nextcloud catalog sync and the duplicate PHPStan/PHPUnit/runner patches, retain only its application-specific Playwright fixture/matrix patch, and replace six.patch.licensefiles with oneREUSE.tomlentry. Do not merge the consumer PR before the shared template change has propagated.This PR is for review; no automatic merge or deploy is requested.