-
Notifications
You must be signed in to change notification settings - Fork 19
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
Migration to metaschema-xslt
#393
Migration to metaschema-xslt
#393
Conversation
@david-waltermire-nist Currently building the site yields the error:
This also happens in |
@nikitawootten-nist I'll look into the website issue. We may want to keep the support/xml and support/json directories, which provide base schema support. Otherwise, I like what you are removing. |
e241ab5
to
2700a7f
Compare
@david-waltermire-nist done. @wendellpiez and I were unsure which repo that content should live, but we see an argument for keeping it here. |
@nikitawootten-nist Would you please rebase to pickup the changes from #398? Thanks. |
2700a7f
to
f6d78bd
Compare
* Removed now irrelevant submodules, dependencies, and scripts * Switched the docs website to use hugo modules
4681d16
to
70df51c
Compare
@nikitawootten-nist I removed the website build changes and moved these to PR #422. This should clear this PR for merging. |
@nikitawootten-nist Have the Test suite xslt helpers moved to |
@nikitawootten-nist Also, if you believe this is ready, can you remove the |
metaschema-xslt
metaschema-xslt
Yep, although the test suite helpers have not been wired yet. Otherwise I think we're ready to merge! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for doing this!
Committer Notes
Prunes content moved to
metaschema-xslt
.metaschema-xslt
- note the test suite has not been integrated yet.All Submissions:
Changes to Core Features: