diff options
| author | John Villalovos <john@sodarock.com> | 2022-06-27 12:53:49 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-06-27 21:53:49 +0200 |
| commit | 212ddfc9e9c5de50d2507cc637c01ceb31aaba41 (patch) | |
| tree | c5a0a92cb72b1886567330381e8ba5a277efb45d /gitlab/utils.py | |
| parent | ebd579588d05966ee66cce014b141e6ab39435cc (diff) | |
| download | gitlab-212ddfc9e9c5de50d2507cc637c01ceb31aaba41.tar.gz | |
refactor: avoid possible breaking change in iterator (#2107)
Commit b6447211754e126f64e12fc735ad74fe557b7fb4 inadvertently
introduced a possible breaking change as it added a new argument
`iterator` and added it in between existing (potentially positional) arguments.
This moves the `iterator` argument to the end of the argument list and
requires it to be a keyword-only argument.
Diffstat (limited to 'gitlab/utils.py')
| -rw-r--r-- | gitlab/utils.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/gitlab/utils.py b/gitlab/utils.py index 6acb861..4d2ec8d 100644 --- a/gitlab/utils.py +++ b/gitlab/utils.py @@ -34,9 +34,10 @@ class _StdoutStream: def response_content( response: requests.Response, streamed: bool, - iterator: bool, action: Optional[Callable], chunk_size: int, + *, + iterator: bool, ) -> Optional[Union[bytes, Iterator[Any]]]: if iterator: return response.iter_content(chunk_size=chunk_size) |