-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(gitlab): add filtering and pagination (#681) #681
Conversation
c43f9a8
to
14bc890
Compare
14bc890
to
c5f79ee
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #681 +/- ##
==========================================
+ Coverage 59.46% 60.06% +0.60%
==========================================
Files 32 32
Lines 3298 3308 +10
==========================================
+ Hits 1961 1987 +26
+ Misses 1337 1321 -16
|
BREAKING CHANGE: The response of `gitlab_projects` API endpoint has been modified to also include the pagination details. Closes reanahub#518
BREAKING CHANGE: The response of `gitlab_projects` API endpoint has been modified to also include the pagination details. Closes reanahub#518
b88f105
to
deae9fa
Compare
BREAKING CHANGE: The response of `gitlab_projects` API endpoint has been modified to also include the pagination details. Closes reanahub#518
deae9fa
to
eac3cbf
Compare
BREAKING CHANGE: The response of `gitlab_projects` API endpoint has been modified to also include the pagination details. Closes reanahub#518
eac3cbf
to
6e413bb
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.
Works nicely 👍
Some cosmetic comments:
-
Left a comment about a (forgotten?) FIXME in the code.
-
The base commit could perhaps say "add" instead of "accept" (?), giving something like:
feat(gitlab): add search query parameter to filter projects (#681)
reana_server/utils.py
Outdated
@@ -509,23 +509,30 @@ def _get_gitlab_hook_id(project_id, gitlab_token): | |||
:param project_id: Project id on GitLab. | |||
:param gitlab_token: GitLab token. | |||
""" | |||
reana_hook_id = None | |||
# FIXME: handle pagination of results |
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.
Do we need the FIXME here?
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.
This is fixed by #684 , but I have removed it anyway given that it will be fixed soon
BREAKING CHANGE: The REST API endpoint `gitlab_projects` now includes pagination details. Closes reanahub#518
6e413bb
to
fb19584
Compare
BREAKING CHANGE: The REST API endpoint `gitlab_projects` now includes pagination details. Closes reanahub#518
fb19584
to
4d23c62
Compare
Updated the commit title, and also the BREAKING CHANGE notice to read as follows: BREAKING CHANGE: The REST API endpoint |
feat(gitlab): add search query parameter to filter projects (feat(gitlab): add filtering and pagination (#681) #681)
feat(gitlab): add support for paginated project list (feat(gitlab): add filtering and pagination (#681) #681)
BREAKING CHANGE: The REST API endpoint
gitlab_projects
now includespagination details.
Closes #518