Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[master < path] Write docs for path module #1018

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

mpintaric55334
Copy link
Contributor

Description

Please briefly explain the changes you made here.

Write docs for path MAGE QM.

Pull request type

Please delete options that are not relevant and check the ones that are.

  • New documentation page

Related PRs and issues

memgraph/mage#313
PR this doc page is related to:
Closes (link to issue):

Checklist:

  • I checked all content with Grammarly
  • I performed a self-review of my code
  • I made corresponding changes to the rest of the documentation
  • The build passes locally
  • My changes generate no new warnings or errors

@mpintaric55334 mpintaric55334 added the status: draft PR is in draft phase label Aug 31, 2023
@mpintaric55334 mpintaric55334 self-assigned this Aug 31, 2023
@mpintaric55334 mpintaric55334 changed the title [master < path] Write docs for path.expand [master < path] Write docs for path module Aug 31, 2023
@imilinovic imilinovic added status: ready PR is ready for review and removed status: draft PR is in draft phase labels Sep 4, 2023
@vpavicic vpavicic added status: ship it PR approved and removed status: ready PR is ready for review labels Sep 5, 2023
@vpavicic vpavicic self-requested a review September 5, 2023 12:42
@ind1xa ind1xa added status: ready PR is ready for review and removed status: ship it PR approved labels Sep 8, 2023
@mpintaric55334
Copy link
Contributor Author

mpintaric55334 commented Sep 8, 2023

@vpavicic Idora added 2 functions which were requested recently by a user, so i re-requested a review

</span>
);

The `path` module is a module for working with intricate connections in graph-like structures. It provides functions and procedures to navigate and analyze paths, helping users uncover meaningful insights from complex data relationships.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The `path` module is a module for working with intricate connections in graph-like structures. It provides functions and procedures to navigate and analyze paths, helping users uncover meaningful insights from complex data relationships.
The `path` module is a module for working with intricate connections in graph-like structures. It provides functions and procedures to navigate and analyze paths, helping users uncover meaningful insights from complex data relationships.

FYI this sounds like a politician wrote it :D You told me nothing :D At this point I have no idea what I will be able to do with this module or is it for me or my use case... I need to read on and check procedures and that's annoying :))

Copy link
Collaborator

@vpavicic vpavicic left a comment

Choose a reason for hiding this comment

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

Maybe you can add more substance to the description of the module?

@vpavicic vpavicic added status: change PR reviewed - needs changes and removed status: ready PR is ready for review labels Sep 8, 2023
@ind1xa ind1xa added status: ready PR is ready for review and removed status: change PR reviewed - needs changes labels Sep 9, 2023
@ind1xa ind1xa requested a review from vpavicic September 9, 2023 07:20
@vpavicic vpavicic added status: ship it PR approved and removed status: ready PR is ready for review labels Sep 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants