Skip to content

Conversation

@dogofbrian
Copy link
Contributor

The examples in Patterns / Syntax and semantics / shortestPaths / Examples mix legacy shortest path and QPPs, which is disallowed. This PR changes them to variable-length relationships.

It also improves the header "Restricted to variable length". QPPs are disallowed but also variable length. What they are not are variable-length relationships.

@neo-technology-commit-status-publisher
Copy link
Collaborator

neo-technology-commit-status-publisher commented Jan 9, 2025

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

@JPryce-Aklundh JPryce-Aklundh added cherry-pick-this-to-5.x Cherry pick this PR changes to the 5.x branch cherry-pick-this-to-cypher-25 labels Jan 9, 2025
@JPryce-Aklundh JPryce-Aklundh merged commit d3adebd into dev Jan 9, 2025
5 checks passed
@JPryce-Aklundh JPryce-Aklundh deleted the shortestpath-examples-fix branch January 9, 2025 15:15
JPryce-Aklundh pushed a commit that referenced this pull request Jan 9, 2025
The examples in Patterns / Syntax and semantics / shortestPaths /
Examples mix legacy shortest path and QPPs, which is disallowed. This PR
changes them to variable-length relationships.

It also improves the header "Restricted to variable length". QPPs are
disallowed but also variable length. What they are not are
variable-length relationships.
JPryce-Aklundh pushed a commit that referenced this pull request Jan 9, 2025
The examples in Patterns / Syntax and semantics / shortestPaths /
Examples mix legacy shortest path and QPPs, which is disallowed. This PR
changes them to variable-length relationships.

It also improves the header "Restricted to variable length". QPPs are
disallowed but also variable length. What they are not are
variable-length relationships.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick-this-to-5.x Cherry pick this PR changes to the 5.x branch dev team-cypher-planner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants