docs: update compatibility matrix#102
Merged
Merged
Conversation
ali-ince
reviewed
Jun 23, 2026
ali-ince
left a comment
ali-ince
left a comment
Contributor
There was a problem hiding this comment.
Looks good, just had two small comments.
ali-ince
approved these changes
Jun 24, 2026
nvitucci
reviewed
Jun 24, 2026
nvitucci
left a comment
nvitucci
left a comment
Contributor
There was a problem hiding this comment.
Before continuing with other comments, I think it would make more sense to either just have only one compatibility table for both Neo4j and Python/Scala versions, or have two tables but in the same page. This will make updates easier in the future, both for introducing new versions and to set a given version as EOL.
nord-erik
force-pushed
the
conn-986-quick-doc-tidy-up-fixes-for-spark
branch
from
235ac75 to
3b84a60
Compare
Contributor
|
In the latest revision, I moved all compatibility tables into one page. Then instead of in-lining them in e.g. installation.adoc I simply reference to it. This creates one page that should hold the single source of truth |
Contributor
|
Please re-review when you have time @nvitucci |
nord-erik
force-pushed
the
conn-986-quick-doc-tidy-up-fixes-for-spark
branch
2 times, most recently
from
26b7bda to
531db6b
Compare
fbiville
commented
Jun 24, 2026
fbiville
commented
Jun 24, 2026
Include an appendix with older versions too.
'nuff said
We have some granular support requirements for spark versions, previous version looked like all 5.x connectors supported all spark.
this fixes the review comments Co-authored-by: Nicola Vitucci <nicola.vitucci@neo4j.com>
Co-authored-by: Florent Biville <445792+fbiville@users.noreply.github.com>
Co-authored-by: Florent Biville <445792+fbiville@users.noreply.github.com>
nord-erik
force-pushed
the
conn-986-quick-doc-tidy-up-fixes-for-spark
branch
from
a76c6db to
1dfb8eb
Compare
nvitucci
requested changes
Jun 24, 2026
nvitucci
left a comment
nvitucci
left a comment
Contributor
There was a problem hiding this comment.
Just a few changes for spelling and clarity
Co-authored-by: Nicola Vitucci <nicola.vitucci@gmail.com>
nvitucci
reviewed
Jun 25, 2026
Co-authored-by: Nicola Vitucci <nicola.vitucci@gmail.com>
nvitucci
approved these changes
Jun 25, 2026
Collaborator
|
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Include an appendix with older versions too.