- Notifications
You must be signed in to change notification settings - Fork675
Closed
Description
Currently theparticipants() method is usinghttp_get().
Should it usehttp_list()? I'm thinking yes.@nejch What do you think?
https://docs.gitlab.com/ee/api/merge_requests.html#get-single-merge-request-participants
python-gitlab/gitlab/mixins.py
Lines 914 to 935 in51d8f88
| defparticipants(self,**kwargs:Any)->Dict[str,Any]: | |
| """List the participants. | |
| Args: | |
| all: If True, return all the items, without pagination | |
| per_page: Number of items to retrieve per request | |
| page: ID of the page to return (starts with page 1) | |
| **kwargs: Extra options to send to the server (e.g. sudo) | |
| Raises: | |
| GitlabAuthenticationError: If authentication is not correct | |
| GitlabListError: If the list could not be retrieved | |
| Returns: | |
| The list of participants | |
| """ | |
| path=f"{self.manager.path}/{self.encoded_id}/participants" | |
| result=self.manager.gitlab.http_get(path,**kwargs) | |
| ifTYPE_CHECKING: | |
| assertnotisinstance(result,requests.Response) | |
| returnresult |
Possibly related is#2912
Metadata
Metadata
Assignees
Labels
No labels