-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Update arguments to parameters (Additional) [1]
#20711
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 5 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
58988cc
Update error message for missing argument type annotation
hyperkai 416a76a
Update error messages for missing type annotations
hyperkai 8c5d9cb
Fix type annotation error messages in tests
hyperkai d29c114
Fix type annotation error message in test
hyperkai a894621
Fix wording in error message for self parameter
hyperkai 5343adb
Update check-classes.test
hyperkai a927084
Update check-selftype.test
hyperkai 1a7ac4f
Update check-overloading.test
hyperkai f5551e0
Update message_registry.py
hyperkai 62f2204
Update check-functions.test
hyperkai 79f7b11
Update message_registry.py
hyperkai 48c627c
Update check-fastparse.test
hyperkai f53a296
Update parse-errors.test
hyperkai 9d0ae67
Update semanal-errors.test
hyperkai 5458f4d
Update check-errorcodes.test
hyperkai 5c8534b
Update message_registry.py
hyperkai 8afa534
Update check-fastparse.test
hyperkai 1755885
Update parse-errors.test
hyperkai 47764f4
Update check-columns.test
hyperkai 30dca1e
Update fine-grained.test
hyperkai 6b9bde7
Update semanal-errors.test
hyperkai 90e6954
Update check-functions.test
hyperkai 4566bd1
Update check-errorcodes.test
hyperkai b9ab2a7
Update message_registry.py
hyperkai 605c8b9
Update message_registry.py
hyperkai 2d825fc
Update fastparse.py
hyperkai 14ac318
Update fastparse.py
hyperkai 24f8fba
Update checker.py
hyperkai cb7277a
Update message_registry.py
hyperkai 6116842
Update message_registry.py
hyperkai db4ac33
Update message_registry.py
hyperkai 5cd99e6
Update check-classes.test
hyperkai 1c7535f
Update check-selftype.test
hyperkai c7e904c
Update check-overloading.test
hyperkai File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also changed
Selftoselfwhich is actually used as a parameter because It's more understandable:There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be more consistent to use
"self" parameter ...if using the lower-case version. I'd prefer it it was either quoted this way (we use double quotes) or unchanged from original, with capitalized 'Self'.