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

Update neo4j dependency #263

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Conversation

alex-linx
Copy link
Contributor

Description

Update the neo4j dependency to be more general, specifically to allow for using in conjuction with neo4j driver v5.

Pull request type

  • Build or packaging related changes

Related issues

Closes #262

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

I tested it against a neo4j running in docker.
Running against neo4j v5.1 did break because the uniqueness keyword doesn't exist anymore in show indexes.
However the neo4j driver v5 is backward compatbile so when running the tests against neo4j docker v4.4 they pass

######################################

Reviewer checklist (the reviewer checks this part)

  • Core feature implementation
  • Tests
  • Code documentation
  • Documentation on memgraph/docs

######################################

@katarinasupe
Copy link
Contributor

Hi @alex-linx thank you for the contribution! The team will probably sync next week to agree on the next release. I will keep you updated.

@katarinasupe katarinasupe mentioned this pull request Aug 30, 2023
4 tasks
@katarinasupe
Copy link
Contributor

Please just remove trigger typing fix from here, so we have it at #260 :)

@alex-linx
Copy link
Contributor Author

@katarinasupe Done

@antepusic antepusic added the status: ship it PR approved label Aug 31, 2023
@katarinasupe
Copy link
Contributor

Thank you @alex-linx for the contributions. We will work on the release at the end of the next week. Stay tuned :)

@katarinasupe katarinasupe merged commit 066690f into memgraph:main Sep 20, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ship it PR approved
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUG] Relax neo4j dependency version
3 participants