Skip to content

Add a Getting Started section to the docs#69

Merged
adamnsch merged 4 commits intomainfrom
getting-started-docs
Jan 24, 2025
Merged

Add a Getting Started section to the docs#69
adamnsch merged 4 commits intomainfrom
getting-started-docs

Conversation

@adamnsch
Copy link
Collaborator

@adamnsch adamnsch commented Jan 23, 2025

Thank you for your contribution to the Graph Data Science Client project.

Before submitting this PR, please read Contributing to the Neo4j Ecosystem.

Make sure:

  • You signed the Neo4j CLA (Contributor License Agreement) so that we are allowed to ship your code in our library
  • Your contribution is covered by tests

@adamnsch adamnsch force-pushed the getting-started-docs branch from f8a3fa5 to f6baeb1 Compare January 23, 2025 14:48
@adamnsch adamnsch changed the title WIP: Add a Getting Started section to the docs Add a Getting Started section to the docs Jan 23, 2025
@adamnsch adamnsch force-pushed the getting-started-docs branch from f6baeb1 to 0d281ca Compare January 23, 2025 14:48
@adamnsch adamnsch marked this pull request as ready for review January 23, 2025 14:49
Copy link
Collaborator

@FlorentinD FlorentinD left a comment

Choose a reason for hiding this comment

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

LGTM just a suggestion to avoid the individual .rst files?

Comment on lines +1 to +2
.. autoclass:: neo4j_viz.Node
:members:
Copy link
Collaborator

Choose a reason for hiding this comment

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

you could also try to use automodule to get all the classes automatically

@adamnsch adamnsch force-pushed the getting-started-docs branch from 0d281ca to 08bd07e Compare January 24, 2025 08:12
@adamnsch
Copy link
Collaborator Author

LGTM just a suggestion to avoid the individual .rst files?

I don't know, I think I like having a separate file for each. Makes it easier to navigate for the user I think? And also easier to refer to

@adamnsch
Copy link
Collaborator Author

I'll merge it for now

@adamnsch adamnsch merged commit a98d01d into main Jan 24, 2025
7 checks passed
@adamnsch adamnsch deleted the getting-started-docs branch January 24, 2025 08:58
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