The Quickstart/Usage section comes quite late after a lengthy intro. I think there's still scope for shortening so that the juicy content is not delayed too long. For example, "Fully supports the Java module system, Can run any Cypher statement" are listed as features. I find they'd rather be "anti-features" if they weren't there, so people are likely to take them for granted even if we don't list them. And so on.
I've left a bunch of points/remarks. They're all marked with a question mark ? so they are easy to find. They need to be addressed/removed before merging.
The Metadata section still needs some love, I couldn't understand it (but I don't have much background on catalogs nor have I done much research).
The nesting of Supported statements in SQL to Cypher translation gets too deep, and level 3 headings in includes are not rendered as headings. I've changed it to +=2, but the issues is still there for the last level.
Some more points:
Quickstart/Usage
section comes quite late after a lengthy intro. I think there's still scope for shortening so that the juicy content is not delayed too long. For example, "Fully supports the Java module system, Can run any Cypher statement" are listed as features. I find they'd rather be "anti-features" if they weren't there, so people are likely to take them for granted even if we don't list them. And so on.?
so they are easy to find. They need to be addressed/removed before merging.Supported statements
in SQL to Cypher translation gets too deep, and level 3 headings in includes are not rendered as headings. I've changed it to+=2
, but the issues is still there for the last level.