added type hints to strings/min_cost_string_conversion.py#2337
Conversation
|
|
||
| def compute_transform_tables(X, Y, cC, cR, cD, cI): | ||
| def compute_transform_tables( | ||
| X: str, Y: str, cC: int, cR: int, cD: int, cI: int |
There was a problem hiding this comment.
Maybe it is just me but I hate these variable names. Uppercase characters are reserved for constants in Python and single letter and double letter variable names are so old school. They look kludgy in modern programming. Why make the caller/reader guess all the time?
There was a problem hiding this comment.
Yes, agree with you. I've corrected that in the new commit.
This reverts commit 971c156.
| first_string: str, | ||
| second_string: str, |
There was a problem hiding this comment.
| first_string: str, | |
| second_string: str, | |
| source_string: str, | |
| destination_string: str, |
Co-authored-by: Christian Clauss <cclauss@me.com>
|
Hey @Sonic0588, TravisCI finished with status TravisBuddy Request Identifier: 6650c3e0-e3bb-11ea-ae1e-a7d25f4a9615 |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
poyea
left a comment
There was a problem hiding this comment.
Thank you for your pull request!🤩
|
It would be even better if you can add a few doctests for this file. Take a look at our Contribution Guidelines and open a new pull request for it, if you want to do so! Thanks. |
…s#2337) * done * add types for local variables * Revert "add types for local variables" This reverts commit 971c156. * rename variables * Update strings/min_cost_string_conversion.py Co-authored-by: Christian Clauss <cclauss@me.com> * rename strings * use flake8 * Update strings/min_cost_string_conversion.py Co-authored-by: Christian Clauss <cclauss@me.com>
…s#2337) * done * add types for local variables * Revert "add types for local variables" This reverts commit 971c156. * rename variables * Update strings/min_cost_string_conversion.py Co-authored-by: Christian Clauss <cclauss@me.com> * rename strings * use flake8 * Update strings/min_cost_string_conversion.py Co-authored-by: Christian Clauss <cclauss@me.com>
Describe your change:
Added static type checking to min_cost_string_conversion.py towards issue #2128.
Checklist:
Fixes: #{$ISSUE_NO}.