Skip to content

XQueryX parser#57

Merged
DrRataplan merged 206 commits intomasterfrom
xqueryx-parser
Oct 30, 2018
Merged

XQueryX parser#57
DrRataplan merged 206 commits intomasterfrom
xqueryx-parser

Conversation

@DrRataplan
Copy link
Contributor

@DrRataplan DrRataplan commented Oct 23, 2018

No description provided.

DrRataplan and others added 29 commits October 23, 2018 11:17
@DrRataplan
[skip ci] Start work on the XQueryX parser
36511a4
@DrRataplan
[skip ci] Continue work on the parser
316d736
@JosVerburg @DrRataplan
Implement more of the prolog
7275860
@DrRataplan
[skip ci] More progress
402e882
@DrRataplan
[skip ci] Another working version
6f72bb9
@JosVerburg @DrRataplan
[skip ci] Parse more properties of the Prolog
ac9fc37
@DrRataplan
[skip ci] Parse functionDecls
53169c7
@DrRataplan
[skip ci] Parse VarDecl
ddf3841
@DrRataplan
[skip ci] Start with the ExprSingle rabithole
a218539
@JosVerburg @DrRataplan
[skip ci] Parse more towards path expression
5cf0542
@DrRataplan
Remove the generated parser from the repository
d65094d 
This will be built when you run `npm install`
@DrRataplan
Add .eslintrc file
d2a5746
@JosVerburg @DrRataplan
[skip ci] Work on path expressions
27d2ecf
@JosVerburg @DrRataplan
[skip ci] improve parsing of path expressions
fb0383c
@DrRataplan
[skip ci] Implement ArrowFunction stuffs
9534ad4
@DrRataplan
[skip ci] Implement the path from PostFixExpr -> Literal / FunctionCall
42b2948
@JosVerburg @DrRataplan
[skip ci] Parse path expressions
637c99d
@JosVerburg @DrRataplan
[skip ci] Add "pathExpr" for relative paths, re-enable the simpleMapExpr
154834f
@DrRataplan
[skip ci] Make functionItemExpr work
6d0e677
@DrRataplan
[skip ci] Fix params / functionBody
0617998
@JosVerburg @DrRataplan
[skip ci] Add wildcard support
0422f7b
@DrRataplan
[skip ci] Implement MapConstructor / ArrayConstructor
cd8e209
@JosVerburg @DrRataplan
[skip ci] Remove extra array wrapping step expressions for a relative…
860289c 
… path expression, only add pathExpr if it is a path expression
@DrRataplan
[skip ci] Implement for/let and mainmodules
56592cc
@DrRataplan
[skip ci] Add IfExpr
7268ee7
@DrRataplan
[skip ci] Clean up and fix schemaimport
452d408
@DrRataplan
[skip ci] Allow building the old parser alongside the new one
f282172
@DrRataplan
[skip ci] fix parent shorthand to be seen as a context item expression
413eb26
@JosVerburg @DrRataplan
[skip ci] Misc fixes, start of XQueryX unit tests
05232ab
@DrRataplan DrRataplan force-pushed the xqueryx-parser branch 2 times, most recently from 0c53aaf to 32d05c9 Compare October 23, 2018 09:20
@DrRataplan DrRataplan changed the title Xqueryx parser XQueryX parser Oct 23, 2018
@DrRataplan DrRataplan force-pushed the xqueryx-parser branch 2 times, most recently from e82474c to f3e3e1b Compare October 23, 2018 09:33
@DrRataplan DrRataplan force-pushed the xqueryx-parser branch 2 times, most recently from bec2794 to 5cee667 Compare October 23, 2018 09:52
@DrRataplan
Copy link
Contributor Author

DrRataplan commented Oct 25, 2018

It seems like the following QT3 tests are new-failing:

Alnilam /tmp/diff » diff -u before after | grep -E "^\+" 
+++ after	2018-10-25 09:55:31.355792683 +0200
+  "comp-attr-bad-name-1",
+  "comp-attr-bad-name-2",
+  "comp-attr-bad-name-3",
+  "comp-attr-bad-name-4",
+  "comp-attr-bad-name-5",
+  "comp-attr-bad-name-6",
+  "comp-attr-bad-name-7",
+  "K2-ComputeConComment-4",
+  "comp-elem-bad-name-2",
+  "comp-elem-bad-name-3",
+  "comp-elem-bad-name-4",
+  "comp-elem-bad-name-5",
+  "comp-elem-bad-name-6",
+  "K2-ConText-5",
+  "K2-DirectConElem-53",
+  "K-FunctionCallExpr-25",

@DrRataplan
Copy link
Contributor Author

DrRataplan commented Oct 30, 2018 •
edited
Loading

The problem with most of these failing tests are dependency related. We are not an XQ10/XQ20/XQ30 engine. We are XQ3.1(+)!

The other tests are about new features, which are expected to bring new failing tests.

@DrRataplan DrRataplan merged commit 2f958d3 into master Oct 30, 2018
@DrRataplan DrRataplan deleted the xqueryx-parser branch October 30, 2018 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bwrrp bwrrp bwrrp approved these changes

@JosVerburg JosVerburg Awaiting requested review from JosVerburg

@ThomasGravekamp ThomasGravekamp Awaiting requested review from ThomasGravekamp

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@DrRataplan @bwrrp @JosVerburg