raulcd commented on code in PR #14533:
URL: https://github.com/apache/arrow/pull/14533#discussion_r1066837521
##########
dev/merge_arrow_pr.py:
##########
@@ -287,7 +311,7 @@ def version_tuple(x):
mainline_versions = [v for v in mainline_versions
if f"maint-{v}" not in maintenance_branches]
default_fix_versions = [
- fix_version_from_branch(x, mainline_versions)
+ fix_version_from_branch(mainline_versions)
for x in merge_branches]
return default_fix_versions[0]
Review Comment:
As we are not using `x` the list comprehension is not needed anymore. We can
update this to be:
```
return fix_version_from_branch(mainline_versions)
```
We can also remove the `merge_branches` variable definition as is not used
anymore and remove it from the signature of the function.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
