Skip to content

Updates to CREATE DATABASE for Cypher 25 #2132

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

Merged
merged 9 commits into from
Apr 3, 2025

Conversation

mnd999
Copy link
Contributor

@mnd999 mnd999 commented Feb 18, 2025

  • Explain how existing store files work, this has never really been well documented
  • Add a different set of options for Cypher 25
  • WAIT is different in Cypher 25.

This is an initial attempt at documenting these changes in Cypher 25, would be good to get some feedback from the docs team on the approach.

* Explain how existing store files work, this has never really been well documented
* Add a different set of options for Cypher 25
* WAIT is different in Cypher 25.
@renetapopova
Copy link
Collaborator

It looks exactly how we thought to document the differences between Cypher 5 and Cypher 25. I haven't looked into any details but the structure looks good to me. Thanks, @mnd999

Copy link
Contributor

@HannesSandberg HannesSandberg left a comment

Choose a reason for hiding this comment

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

lgtm 👍 Nice with the cypher versions tabs :)

Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

Some editorial suggestions.

@renetapopova renetapopova self-requested a review February 21, 2025 09:32
Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

Sorry, I clicked approved instead of Comment 😄

Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

@mnd999, could you be able to take a look at my comments? Thanks!

mnd999 and others added 2 commits February 21, 2025 15:23
Co-authored-by: Reneta Popova <reneta.popova@neo4j.com>
@mnd999
Copy link
Contributor Author

mnd999 commented Feb 21, 2025

@renetapopova have done so. The remaining comments are in @nick-giles-neo bits.

Co-authored-by: Reneta Popova <reneta.popova@neo4j.com>
Copy link
Collaborator

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

Thank you both!

@mnd999 mnd999 force-pushed the dev-create-database-cypher25 branch 2 times, most recently from d2e13bf to 3622eb0 Compare February 25, 2025 15:45
@neo-technology-commit-status-publisher
Copy link
Collaborator

This PR includes documentation updates
View the updated docs at https://neo4j-docs-operations-2132.surge.sh

Updated pages:

@renetapopova renetapopova removed their assignment Apr 2, 2025
@renetapopova
Copy link
Collaborator

Hey @mnd999, I have created a new branch in the docs-operations repo called cypher-25. Could you please merge this PR into it? This way, we’ll have fewer conflicts to resolve later and always be able to keep the new content that comes in sync with the dev branch. Thank you so much!

@mnd999 mnd999 changed the base branch from dev to cypher-25 April 3, 2025 13:21
@mnd999 mnd999 merged commit 51fec9e into neo4j:cypher-25 Apr 3, 2025
12 checks passed
@mnd999
Copy link
Contributor Author

mnd999 commented Apr 3, 2025

@renetapopova done

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.

7 participants