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

Small fixes #749

Merged
merged 1 commit into from
Sep 27, 2023
Merged

Small fixes #749

merged 1 commit into from
Sep 27, 2023

Conversation

gem-neo4j
Copy link
Contributor

I found some issues with the formatting and links, so fixed them up!

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

Thanks for the documentation updates.

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

Copy link
Contributor

@JPryce-Aklundh JPryce-Aklundh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @gem-neo4j!

@@ -1499,7 +1499,7 @@ With this in mind, the below list contains all supported types (as of Neo4j 5.13

This should be taken into account when relying on the output of the `valueType()` function.

See the xref::syntax/expressions.adoc#type-predicate-expressions[type predicate expression] for an alternative way of testing type values.
See the xref::values-and-types/type-predicate.adoc[type predicate expression] for an alternative way of testing type values.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be plural as well, right? ("type predicate expressions")

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well it is just one expression though 🤔 Would we say "see the Count Expression" or "see the Count Expressions"? :)

@JPryce-Aklundh JPryce-Aklundh merged commit deeb032 into neo4j:dev Sep 27, 2023
2 checks passed
JPryce-Aklundh pushed a commit to JPryce-Aklundh/docs-cypher that referenced this pull request Oct 23, 2023
I found some issues with the formatting and links, so fixed them up!
@JPryce-Aklundh JPryce-Aklundh mentioned this pull request Oct 23, 2023
renetapopova added a commit that referenced this pull request Oct 23, 2023
Includes the following PRs:

- #676
- #687
- #688
- #722
- #726
- #734
- #735
- #737
- #748
- #749
- #752
- #766

---------

Co-authored-by: Gem Lamont <[email protected]>
Co-authored-by: Nils Ceberg <[email protected]>
Co-authored-by: Matthew Parnell <[email protected]>
Co-authored-by: Matthew Parnell <[email protected]>
Co-authored-by: Reneta Popova <[email protected]>
Co-authored-by: Lasse Heemann <[email protected]>
Co-authored-by: Ali Ince <[email protected]>
Co-authored-by: Therese Magnusson <[email protected]>
Co-authored-by: Stefano Ottolenghi <[email protected]>
Co-authored-by: Therese Magnusson <[email protected]>
Co-authored-by: linneaandersson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants