Skip to content

Conversation

@rvdb
Copy link
Contributor

@rvdb rvdb commented Oct 2, 2025

…otherwise they would be skipped from the resulting compiled XSLT version used for validation in Oxygen)

This sch:pattern wrapper had been removed in subsequent commits.

…otherwise they would be skipped from the resulting compiled XSLT version used for validation in Oxygen)
@rvdb rvdb requested a review from sydb October 2, 2025 08:45
@rvdb
Copy link
Contributor Author

rvdb commented Oct 2, 2025

@sydb This is the proper resolution for preserving global Schematron variables, IMO. The other suggestion I made, adjusting teiodds.xsl so it would wrap <tei:constraintDecl> child elements in <sch:pattern> in the resulting RNG output would be too broad. When testing locally, it appeared the tei:constraintDecl/sch:ns declarations from the ODD file would be invalid when transformed to sch:pattern/sch:ns in the resulting RNG file. Hence this tweak to the Schematron fragments in the ODD file, which solves the issue.

Copy link
Contributor

@martindholmes martindholmes left a comment

Choose a reason for hiding this comment

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

I wonder if the best way to get the current TEI version is from Jenkins. Shouldn't there be somewhere on the main website to get this info?

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