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

XQuery 3.1 Map Syntax #77

Open
jmeischner opened this issue Feb 7, 2018 · 8 comments
Open

XQuery 3.1 Map Syntax #77

jmeischner opened this issue Feb 7, 2018 · 8 comments

Comments

@jmeischner
Copy link

It seems that xqlint does not support the map syntax of XQuery 3.1.

19 |function _:schema-render-table-prepare-rows-jf($Items as element()*, $Schema as element(schema),$Context as map(*))
	                                                                                                                   ^ lexical analysis failed
while expecting '(:'
after successfully scanning 1 characters beginning at line 19, column 112:
...(*))

Is it possible to add this feature to xqlint?

@joewiz
Copy link
Contributor

joewiz commented Feb 7, 2018

This feature was added in @wolfgangmm's fork for eXist: wolfgangmm@2b0479d.

@wcandillon
Copy link
Owner

@joewiz is there a feature you would like to see merge in the main package? Let me know.

@jmeischner
Copy link
Author

Thank you for your answer. Is there a reason that this feature is not merged back into this repo?

I created the sublime linter package for this linter and it's much easier to install it simply by using npm than it seem to install the forked one.

@jmeischner
Copy link
Author

Is there anything I can do to push this topic?

@joewiz
Copy link
Contributor

joewiz commented Mar 29, 2018

@wcandillon I would certainly support the idea of merging this feature into the main xqlint.

@jmeischner I'm sure @wolfgangmm would be happy to see the feature pulled into the main xqlint project. Would you be able to prepare a PR?

@wcandillon
Copy link
Owner

Yes please guys go for it! Definitely would love to see it in the main xqlint. I even had a proof of concept for this at #48

@jmeischner
Copy link
Author

@joewiz Unfortunately I don't think that I'm the right one to prepare a PR because I do not know where to start to accomplish this. I am not really experienced with the GitHub mechanisms =(

@RdNetwork
Copy link

Hi,
Any news on this?
Was this ever added to xqlint?

I'm using the XML Tools for Visual Studio Code extension, which seems to be based on this project, and I run into the same issue.

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

No branches or pull requests

4 participants