Skip to content
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

Purge trailing whitespace in docs source files #7930

Merged

Conversation

paultcochrane
Copy link
Contributor

@paultcochrane paultcochrane commented Jul 28, 2022

Trailing whitespace is forbidden in source code for this project
(implemented as part of the PEP8 checks), however the documentation
source files contained trailing whitespace. It seemed appropriate to
remove trailing whitespace in the documentation source files as well.

This PR contains only whitespace changes; there are no content changes. If you would like something done differently here, please let me know and I'll be more than hapy to update the PR and resubmit as necessary.

Maintainer merge checklist

  • Title is descriptive/clear for inclusion in release notes.
  • Applied a Component: xxx label.
  • Applied the api-deprecation or api-break label.
  • Applied the release-highlight label to be highlighted in release notes.
  • Added to the milestone version it was merged into.
  • Unittests are included in PR.
  • Properly documented, including versionadded, versionchanged as needed.

Trailing whitespace is forbidden in source code for this project
(implemented as part of the PEP8 checks), however the documentation
source files contained trailing whitespace.  It seemed appropriate to
remove trailing whitespace in the documentation source files as well.
@tshirtman
Copy link
Member

tshirtman commented Jul 28, 2022

Thanks, just to be sure, did you check this had no impact on the rendering of the affected blocks? i’m pretty sure it shouldn’t, but i didn’t touch RST in some time and wouldn’t want such a change to have unforseen visible effects :).

@paultcochrane
Copy link
Contributor Author

Yes, I checked that the rendering still looked ok. It wasn't that easy to check everything as thoroughly as I'd have liked, but anywhere that I thought the rendering might have changed, everything still looked ok. Would you like me to have another look to check for certain?

@tshirtman tshirtman merged commit ce0bf8c into kivy:master Oct 2, 2022
@tshirtman
Copy link
Member

Sorry for lack of answer, thanks again!

@misl6 misl6 added the Component: documentation Documentation or examples label Oct 2, 2022
@misl6 misl6 added this to the 2.2.0 milestone Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: documentation Documentation or examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants