wcandillon / xqlint

JSONiq & XQuery Quality Checker
Apache License 2.0
52 stars 22 forks source link

XQuery 3.1 Map Syntax #77

Open jmeischner opened 6 years ago

jmeischner commented 6 years ago

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 commented 6 years ago

This feature was added in @wolfgangmm's fork for eXist: https://github.com/wolfgangmm/xqlint/commit/2b0479d9007b32de173194c8a5c39fad50a5f483.

wcandillon commented 6 years ago

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

jmeischner commented 6 years ago

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 commented 6 years ago

Is there anything I can do to push this topic?

joewiz commented 6 years ago

@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 commented 6 years ago

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 https://github.com/wcandillon/xqlint/pull/48

jmeischner commented 6 years ago

@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 commented 3 years ago

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.