-
Notifications
You must be signed in to change notification settings - Fork 959
feat: add dependency list export client service #2063
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lmphil thanks for the contribution! I did a first round of review and left a couple of comments for you - back to you 🏓
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lmphil thanks for the changes! I have some more additions - back to you 🏓
@timofurrer Done and tested. Thanks for the feedback. 🏓 |
faa3a8a
to
a7ae7bd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lmphil I've left one last suggestion - then we are good for the merge.
Thanks for pulling through 🤝
I've actually applied the patch myself and will merge :) |
This PR adds support for the GitLab Dependency List Export API:
https://docs.gitlab.com/ee/api/dependency_list_export.html