Skip to content

Add check-structure.php, comparing translation structure at declared revision - #342

Merged
lacatoire merged 1 commit into
php:masterfrom
lacatoire:translation/check-structure
Sep 10, 2026
Merged

Add check-structure.php, comparing translation structure at declared revision#342
lacatoire merged 1 commit into
php:masterfrom
lacatoire:translation/check-structure

Conversation

@lacatoire

@lacatoire lacatoire commented Sep 9, 2026

Copy link
Copy Markdown
Member

Compares the block structure of translated files with doc-en: the elements and their nesting, not the prose they contain.

Six translations already run it from their own copy, in three variants; one of them never received the fix that added href and xpointer to the compared attributes, and lets a translated XInclude target through.

Checked against doc-fr: 11359 files, 6 differences, the same ones its local copy reports.

First consumer: php/doc-fr#3544.

…revision

The qaxml-* scripts compare a translation with the current doc-en file, so a
file waiting for a sync reports differences that are only lag. This one reads
the revision tag of each file and compares with doc-en at that revision, so
those files produce no alert and the check can block a pull request.

Six translations carry a copy of this script in their own .github/scripts
directory, in three variants; one of them never received the fix that added
href and xpointer to the compared attributes, and lets a translated XInclude
target through.
@lacatoire
lacatoire force-pushed the translation/check-structure branch from b4b2b8e to fa1dfd0 Compare September 9, 2026 12:14
@lacatoire

Copy link
Copy Markdown
Member Author

Corrected PL build here php/doc-pl#36

@alfsb alfsb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for keeping the uncommon code style.

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.

2 participants