eumiro commented on code in PR #33479:
URL: https://github.com/apache/airflow/pull/33479#discussion_r1299301929
##########
scripts/ci/pre_commit/pre_commit_check_deferrable_default.py:
##########
@@ -76,7 +76,7 @@ def iter_check_deferrable_default_errors(module_filename:
str) -> Iterator[str]:
args = node.args
arguments = reversed([*args.args, *args.kwonlyargs])
defaults = reversed([*args.defaults, *args.kw_defaults])
- for argument, default in itertools.zip_longest(arguments, defaults,
fillvalue=None):
+ for argument, default in zip(arguments, defaults):
Review Comment:
In the subsequent line:
```python
if argument is None or default is None:
continue
```
all the lines that come from `zip_longest` but are ignored in `zip` are
ignored anyway.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
