-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Capitalize error messages consistently #15125
Comments
In the above PR, I left some errors as they are, specifically if they reference an identifier or if they are part of a comment. Here are the errors left intact, found by searching with regex
|
Perhaps these should be quoted, e.g. |
That sounds reasonable, but would we need to quote identifiers like that if they appear after the first word of the error? |
Most error messages start with a capital letter, but syntax errors in particular start with a lowercase letter. Example program:
Error message:
The desired message would be like this:
You can search for the regexp
# E: [a-z]
undertest-data/
to find most of these inconsistent error messages. We should make the error messages consistent.Note that some error messages are from the Python
ast
module which we use for parsing. We can probably just capitalize the first letter of all of these errors.The text was updated successfully, but these errors were encountered: