Skip to content

CI: use the shared check-structure.php from doc-base - #3544

Merged
lacatoire merged 3 commits into
php:masterfrom
lacatoire:translation/shared-check-structure
Sep 10, 2026
Merged

CI: use the shared check-structure.php from doc-base#3544
lacatoire merged 3 commits into
php:masterfrom
lacatoire:translation/shared-check-structure

Conversation

@lacatoire

@lacatoire lacatoire commented Sep 9, 2026

Copy link
Copy Markdown
Member

Replaces the local copy of check-structure.php with the shared one added in php/doc-base#342, so that every translation runs the same version. The copy here and the ones in doc-es, doc-it, doc-pt_br, doc-ru and doc-zh have already drifted into three variants.

The workflow now checks out fr, en and doc-base side by side, the layout the doc-base scripts expect and the one the build workflow already uses. The alerts move from French to English, like every other doc-base script.

Also migrates check-en-revision.yml to the shared check-en-revision.php script added in php/doc-base#343. That step points at the PR branch until php/doc-base#343 is merged.

The script is identical in six translation repositories, in three variants,
and one of them lags behind a fix. Calling the doc-base copy keeps every
translation on the same version.

The workflow now checks out fr, en and doc-base side by side, the layout the
doc-base scripts expect and the one the build workflow already uses.
To be dropped once that pull request is merged, so that the workflow tracks
doc-base master.
@lacatoire
lacatoire marked this pull request as ready for review September 9, 2026 12:03
@lacatoire
lacatoire merged commit beb186b into php:master Sep 10, 2026
5 checks passed
@lacatoire
lacatoire deleted the translation/shared-check-structure branch September 10, 2026 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant