Skip to content

Conversation

@alanjds
Copy link

@alanjds alanjds commented Oct 6, 2018

Rebased @jamdagni86 work on google#265 over #87

Kudos @jamdagni86 for the code and @trotterdylan for original comments.

@alanjds alanjds requested review from a team, cclauss and corona10 October 6, 2018 19:15
else:
args.append(arg.arg)
# args = [a.arg for a in node_args.args]

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

args = [arg.elts if isinstance(arg, ast.Tuple) arg.arg for arg in node_args.args]

Copy link
Author

@alanjds alanjds Oct 8, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

args = [(arg.elts if isinstance(arg, ast.Tuple) else arg.arg) for arg in node_args.args]

I am wondering if this version is more readable than the original 5-line-long one

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's just the pythonic way, but you'r right . the original is more readable.

@alanjds alanjds merged commit e353540 into master Oct 22, 2018
@alanjds
Copy link
Author

alanjds commented Oct 22, 2018

Thanks @jamdagni86.

@jamdagni86
Copy link

you're welcome @alanjds 👍

@cclauss cclauss deleted the jamdagni86-fix-17 branch October 24, 2018 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants