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

xsi:type currently requires a namespace, resulting in undefined if no… #1159

Merged
merged 1 commit into from
Aug 31, 2021

Conversation

newmaniese
Copy link
Contributor

… XMLNS is defined. Making attributes working without namespace or xmlns definitions.

@jsdevel
Copy link
Collaborator

jsdevel commented Aug 27, 2021

@newmaniese please fix the tests

@newmaniese
Copy link
Contributor Author

Updated, should be passing now.

Sorry that it didn't pass the first time, Oracle's servers were down for the remote XSD, and I didn't realize it bailed on the first failure.

@jsdevel
Copy link
Collaborator

jsdevel commented Aug 30, 2021

@newmaniese build is still failing. can you take a look?

@newmaniese
Copy link
Contributor Author

Looking, thanks

@newmaniese
Copy link
Contributor Author

Managed to not push my changes. Please run it again and thank you.

@jsdevel
Copy link
Collaborator

jsdevel commented Aug 31, 2021

@newmaniese the build is still failing. please run these on your local machine.

… XMLNS is defined. Making attributes working without namespace or xmlns definitions.
@newmaniese
Copy link
Contributor Author

Tests were/are passing and missed a lint command. Fixed all good. Thanks

@jsdevel jsdevel merged commit 6278bad into vpulim:master Aug 31, 2021
@newmaniese
Copy link
Contributor Author

Thanks, sorry for the back and forth!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants