-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update all XSLTs to XSLT 3.0 #119
Comments
Updating to version 3.0 would make the XSLTs less reusable by other systems though, wouldn't it? |
But reusability is exactly one of the goals of the packaging feature. Are our XSLTs currently used in other systems? |
I haven't looked at the packaging feature. And I don't use the modules the way we package them now. But I've sometimes reused (i.e. copy-pasted) some bits from here and there in other projects. I'm worried that might not always be possible if we start using too many XSLT 3.0 features, but maybe it's not really a problem. You probably don't intent to rewrite all XSLTs using non-backwards-compatible features :) All-in-all it's probably good to update to 3.0 👍 |
Not immediately no. |
The new packages feature allows us to have better encapsulation and a clearer API definition (which makes for better documentation and allows us to do semantic versioning). Updating to XSLT 3.0 in itself shouldn't be a big deal. We just need to update Saxon and Calabash (#117) and all 2.0 stylesheets should already be forward compatible with 3.0 (I assume). Adding packages (and other 3.0 features) can be done ad hoc.
The text was updated successfully, but these errors were encountered: