API: add /node and /edge endpoints to check for node/edge existence (!60) · Merge requests · Platform / Development / swh-graph · GitLab

Snippets Groups Projects

Closed Stefano Zacchiroli requested to merge generated-differential-D2380-source into master 5 years ago

Test Plan

tox

Migrated from D2380 (view on Phabricator)

Activity

Software Heritage Continuous Integration for public repositories @swh-public-ci · 5 years ago

Build is green See https://jenkins.softwareheritage.org/job/DGRPH/job/tox/114/ for more details.
Stefano Zacchiroli @zack · 5 years ago

Author Maintainer
API doc: untabify
Software Heritage Continuous Integration for public repositories @swh-public-ci · 5 years ago

Build is green See https://jenkins.softwareheritage.org/job/DGRPH/job/tox/115/ for more details.
Antoine Pietri @seirl · 5 years ago
I think the API could be more REST:

/node/<src> that gives you the list of neighbors of src (and that you can HEAD to know if it's present or not)

/edge/<src>/<dst> is already REST

We could merge /neighbors like that.
Antoine Pietri @seirl · 5 years ago

Merge request was returned for changes
Stefano Zacchiroli @zack · 5 years ago

Author Maintainer

Updating !60 (closed): API: add /node and /edge endpoints to check for node/edge existence
Software Heritage Continuous Integration for public repositories @swh-public-ci · 5 years ago

Build is green See https://jenkins.softwareheritage.org/job/DGRPH/job/tox/119/ for more details.
vlorentz @vlorentz · 2 years ago

Maintainer

Closing this MR, as it would need a major rewrite before being merged.
vlorentz closed 2 years ago

closed

Please register or sign in to reply