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
Improve style consistency in root directory docs files #4493
Conversation
@@ -1,4 +1,4 @@ | |||
# Github actions template config | |||
# Github actions template config. |
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.
A lot of those are just because, for example 4 lines below, the dot is used.
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.
Btw. all these values not intended to be changed at runtime, i.e. python "constants", should be capitalized.
windows = True | ||
else: | ||
windows = False | ||
windows = len(sys.argv) > 1 and "win" in sys.argv[1] # Special case windows to 0.1.6225 |
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.
iirc this is the only change that's not in a doc or echo statement.
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.
Thanks for cleaning up some of my code as well 😉
Fixed the black test, was about removing a blank. |
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.
aok
Read through the files in the root directory and where I saw that various conventions where mixed or sentences could be completed, made some edits. When in doubt I opted for the "more full sentence" conventions with dots at the end and all.
Some hyperlinks are added too.