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

Dev test cypher versions docker #727

Merged
merged 3 commits into from
Feb 4, 2025
Merged

Conversation

gem-neo4j
Copy link
Contributor

Add ability to change Cypher version in Docker Tests

@gem-neo4j
Copy link
Contributor Author

@gem-neo4j gem-neo4j force-pushed the dev_test_cypher_versions_docker branch 4 times, most recently from 1cbc594 to a3f1c85 Compare February 3, 2025 14:25
? System.getenv()
.getOrDefault(
"CYPHER_VERSION",
Util.getCypherVersion(GraphDatabaseInternalSettings.CypherVersion.Cypher25))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I will convert this back to 5 before merge, but wanted to show the tests are all fixed for Cypher 25 :D

@gem-neo4j gem-neo4j marked this pull request as ready for review February 3, 2025 14:59
Copy link
Contributor

@loveleif loveleif left a comment

Choose a reason for hiding this comment

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

Great! There's a merge conflict with https://github.com/neo-technology/neo4j/pull/29353, but I assume you will merge first so I get to handle that :).

@gem-neo4j gem-neo4j force-pushed the dev_test_cypher_versions_docker branch from a3f1c85 to 6a2b121 Compare February 4, 2025 09:47
@gem-neo4j gem-neo4j merged commit 4d2910d into dev Feb 4, 2025
11 checks passed
@gem-neo4j gem-neo4j deleted the dev_test_cypher_versions_docker branch February 4, 2025 10:49
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.

2 participants