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

[fix] some xqlint errors are not displayed #529

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

marmoure
Copy link
Contributor

@marmoure marmoure commented Sep 26, 2022

fixes #224
@joewiz i opened a new PR for #224
image

This open source contribution to the eXide project was commissioned by the Office of the Historian, U.S. Department of State, https://history.state.gov/.

@dizzzz dizzzz requested a review from joewiz October 3, 2022 18:26
@joewiz
Copy link
Member

joewiz commented Oct 4, 2022

@marmoure I have tested this and can see it surfaces the xqlint errors. As we discussed this isn't a full fix to #224 - that requires updating xqlint to accept valid XQuery 3.1 syntax. But it is a step forward while you work on the xqlint aspect, so I'll merge this now.

@marmoure
Copy link
Contributor Author

marmoure commented Jan 4, 2023

@marmoure i need to re open this PR after it's reverted as a draft for now

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

Successfully merging this pull request may close these issues.

xqlint error causes "run test" to silently fail
3 participants