-
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
Add support for validating SDMX-ML v2.1 messages #153
Conversation
@goatsweater many thanks for the contribution here! The test failures are due to added specimens for which the support is in #152. So for next steps:
…and then we will go from there. I will also investigate the settings for this repo so I don't need to manually approve the CI runs on every push you (or other contributors) make to your branch. |
@khaeru I saw that you had made some changes since my fork and merged those into this branch. I think that may have messed up the ability to rebase. I'm happy to close this and start over with a proper rebase if you'd like. |
If you don't mind, that would be appreciated. Please let me know if you get stuck and I can try to create a new branch by cherry-picking some commits from this one. In the meantime I'll look at the CLA setup, as promised. |
Closing in favor of #154 |
Required: write a single sentence that describes the changes made by this PR.
Add support for validating SDMX-ML messages via
sdmx.validate_xml()
(Closes #51)PR checklist