-
Notifications
You must be signed in to change notification settings - Fork 343
Improve and fix docs of GUI system #1480
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
Conversation
Fixed some documentation and made it neater. - Maximum character width: 80 chars - Capitalize comments - Replace "min" with "minimum" - Add RST formatting to code blocks and sections - Add newlines where they are needed I am not that familiar with ReStructuredText. Double check that the multiline parameter docs and the list docs display correctly.
I'll add some more commits soon; don't merge immediately. |
Just checking, you can see what the auto-checks are complaining about? |
@pvcraven there might be some multiline code stuff that messed it up. If we had a line of code like:
Then I'm not sure how to wrap it to eighty characters. For now I have only used the blackslash character |
You can see what it complains about here: Run flake8 on your code base to see the issues locally. |
@eruvanos Is this PR salvagable? |
Yes, we only have to fix the format for failing pep8. |
In addition to the CI failures, many of the changes in this PR break links to classes and break PEP-257:
Given the original poster (@eschan145) has not responded for a long time, should this PR be considered abandoned and ok to fork / update? |
Merged as #1670 |
No description provided.