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

Fix for issue #68 #77

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@xml-project
Contributor

xml-project commented Oct 12, 2018

Not sure I found all, but surely got most of them.
The problem was: Namespace declarations were removed from p:declare-step, so they were not defined for e.g. <p:wrap-sequence wrapper="c:result" /> or <p:declare-step type="x:step" />.

May be there are some I have misted in the tests expected to fail. But we will find them as time goes by.

@xml-project xml-project requested a review from ndw Oct 12, 2018

@ndw

This comment has been minimized.

Contributor

ndw commented Oct 12, 2018

Yes, we'll find them as we go. Sorry again about botching the namespaces.

@ndw

ndw approved these changes Oct 12, 2018

(When it passes Travis 😄 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment