community icon indicating copy to clipboard operation
community copied to clipboard

Purge trailing whitespace in docs source files

Open paultcochrane opened this issue 2 years ago • 2 comments

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.

paultcochrane avatar Jul 28 '22 12:07 paultcochrane

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 :).

tshirtman avatar Jul 28 '22 14:07 tshirtman

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?

paultcochrane avatar Jul 28 '22 15:07 paultcochrane

Sorry for lack of answer, thanks again!

tshirtman avatar Oct 02 '22 18:10 tshirtman