Skip to content

Commit cbc304c

Browse files
committed
Fix mypy
Turns out multi-line ignores aren't a thing until Python 3.8 [1]. I've no idea how this was passing beforehand. The fix is easy - change the order of the args, allowing us to place things where they need to go. [1] python/mypy#500 (comment) Signed-off-by: Stephen Finucane <[email protected]>
1 parent 8ecba27 commit cbc304c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

git_pw/api.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -143,9 +143,9 @@ def get(url, params=None, stream=False):
143143
# TODO(stephenfin): We only use a subset of the types possible for
144144
# 'params' (namely a list of tuples) but it doesn't seem possible to
145145
# indicate this
146-
rsp = requests.get( # type: ignore
147-
url, auth=_get_auth(), headers=_get_headers(), params=params,
148-
stream=stream)
146+
rsp = requests.get(
147+
url, auth=_get_auth(), headers=_get_headers(), stream=stream,
148+
params=params) # type: ignore
149149
rsp.raise_for_status()
150150
except requests.exceptions.RequestException as exc:
151151
_handle_error('fetch', exc)

0 commit comments

Comments
 (0)