Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use <constraintDecl> in Guidelines and tei_customization, and add CMC to French Guidelines #2660

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

sydb
Copy link
Member

@sydb sydb commented Jan 18, 2025

Three changes:

  • Use <constraintDecl> for Schematron namespace declarations in tei_customization
  • Use <constraintDecl> (instead of hack <constraintSpecs>) in the Guidelines (both en & fr)
  • Added CMC to fr Guidelines (kinda embarrassing that it wasn’t there!)

WARNING:
This PR will not work against the released or current dev Stylesheets, but rather against the sydb_fix_constraintDecl_oversight_01 branch of the Stylesheets; and thus it should not be merged until (TEIC/Stylesheets#727) is merged!

 * Use constraintDecl for Schematron namespace declarations in tei_customization
 * Use constraintDecl (instead of hack constraintSpecs) in the Guidelines (both en & fr)
 * Added CMC to fr Guidelines
@martindholmes
Copy link
Contributor

Looks good to me, but why do we still need the old Schematron namespace?

@sydb
Copy link
Member Author

sydb commented Jan 19, 2025

Good question. Not at all sure that we do. (I just copied the namespace decls that we had.)

@martindholmes
Copy link
Contributor

It occurs to me to wonder what the Stylesheets will do with a chain in which some ODDs use the old declare-it-wherever-you-want approach to ns declaration and the new constraintDecl mechanism. @sydb Do you know?

@martindholmes
Copy link
Contributor

ATOP group tested this today, and everything works as intended, so we're good to go.

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.

Good to go.

@raffazizzi raffazizzi merged commit 6c4ac00 into dev Jan 22, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants