Skip to content
This repository has been archived by the owner on Apr 10, 2024. It is now read-only.

[*.py] Rename "Arguments:" to "Args:" #271

Merged

Conversation

SamuelMarks
Copy link
Contributor

I've written custom parsers and emitters for everything from docstrings to classes and functions. However, I recently came across an issue with the TensorFlow codebase: inconsistent use of Args: and Arguments: in its docstrings. It is easy enough to extend my parsers to support both variants, however it looks like Arguments: is wrong anyway, as per:

Therefore, only Args: is valid. This PR replaces them throughout the codebase.

PS: For related PRs, see tensorflow/tensorflow/pull/45420

@ludwigschubert
Copy link
Contributor

Thanks, Human-Samuel or Script-Samuel! :-)

@ludwigschubert ludwigschubert merged commit 0d81887 into tensorflow:master Mar 12, 2021
@SamuelMarks
Copy link
Contributor Author

You're welcome!

BTW: I did the first one by hand and the rest via fastmod, fd and hub 🤫

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants