Skip to content

Conversation

@JPryce-Aklundh
Copy link
Collaborator

No description provided.

@JPryce-Aklundh JPryce-Aklundh changed the title add instructions for changes in language and gql Update README to include documenting changes to language and GQL Nov 5, 2024
README.adoc Outdated
* xref:README.adoc#cypher-gql[]
[[build-locally]]
Copy link
Contributor

Choose a reason for hiding this comment

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

not sure these have to be set
asciidoc generates anchors for headings by default - it should be #building-locally for the line below

edit: just noticed, on github, both anchors are present now (eg https://github.com/JPryce-Aklundh/docs-cypher/tree/updated_readme?tab=readme-ov-file#documenting-changes and https://github.com/JPryce-Aklundh/docs-cypher/tree/updated_readme?tab=readme-ov-file#documenting-changes-to-cypher)

README.adoc Outdated
Comment on lines 90 to 91
[role=label--new-5.x]
\== Header
Copy link
Contributor

Choose a reason for hiding this comment

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

README.adoc Outdated
Comment on lines 93 to 94
[role=label--deprecated]
\== Header
Copy link
Contributor

Choose a reason for hiding this comment

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

here too

Comment on lines +98 to +99
`featureX` label:new[Introduced in 5.x] +
`featureY` label:deprecated[]
Copy link
Contributor

Choose a reason for hiding this comment

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

these work, but i guess they could use a source block as well

README.adoc Outdated
In these cases, it is often preferable to note the change in a sentence.
For example: "As of Neo4j 5.x, featureX supports ..."

Removed features should be removed from the Cypher Manual.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm a fan of imperative mood for this (usually shorter)
"remove removed ..." isn't exactly elegant though :D

README.adoc Outdated

* If the feature is part of GQL's mandatory features, it should be recorded on the page xref:modules/ROOT/pages/appendix/gql-conformance/supported-mandatory.adoc[].
* If the feature is part of GQL's optional features, it should be recorded on the page xref:modules/ROOT/pages/appendix/gql-conformance/supported-optional.adoc[].
* if the feature adds functionality for which there exists an analogous GQL equivalent, it should be recorded on the page xref:modules/ROOT/pages/appendix/gql-conformance/analogous-cypher.adoc[].
Copy link
Contributor

Choose a reason for hiding this comment

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

"analogous GQL equivalent" - is it analogous or equivalent? their use might be synonymous in this context, but then we only need one 🤔

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

neo-technology-commit-status-publisher commented Nov 7, 2024

Thanks for the documentation updates.

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

@JPryce-Aklundh JPryce-Aklundh merged commit b4fc0fe into neo4j:dev Nov 7, 2024
5 checks passed
@JPryce-Aklundh JPryce-Aklundh deleted the updated_readme branch November 7, 2024 09:43
JPryce-Aklundh added a commit to JPryce-Aklundh/docs-cypher that referenced this pull request Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants