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

65: Allow xmlns="xxx" to NOT change the default namespace for NameTests #753

Merged

Conversation

michaelhkay
Copy link
Contributor

Fix issue #65. Basically, fix the bug whereby xmlns="xxx" changes the default namespace for element NameTests, while retaining bug-compatibility.

@ChristianGruen ChristianGruen changed the title Allow xmlns="xxx" to NOT change the default namespace for NameTests 65: Allow xmlns="xxx" to NOT change the default namespace for NameTests Oct 16, 2023
@ChristianGruen ChristianGruen added the Tests Needed Tests need to be written or merged label Oct 24, 2023
@ChristianGruen ChristianGruen added the Blocked PR is blocked (has merge conflicts, doesn't format, etc.) label Oct 24, 2023
@michaelhkay michaelhkay removed the Blocked PR is blocked (has merge conflicts, doesn't format, etc.) label Oct 24, 2023
@michaelhkay
Copy link
Contributor Author

Resolved conflicts (as usual, these were in the "changes" appendix)

@ChristianGruen ChristianGruen added the Blocked PR is blocked (has merge conflicts, doesn't format, etc.) label Oct 31, 2023
@michaelhkay michaelhkay force-pushed the Issue65-xquery-default-namespace branch from f182064 to 846b7e2 Compare October 31, 2023 19:11
@michaelhkay michaelhkay removed the Blocked PR is blocked (has merge conflicts, doesn't format, etc.) label Oct 31, 2023
@michaelhkay michaelhkay merged commit d8ae4f7 into qt4cg:master Oct 31, 2023
2 checks passed
@michaelhkay michaelhkay deleted the Issue65-xquery-default-namespace branch October 3, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tests Needed Tests need to be written or merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants