Trim trailing whitespaces and update pre-commit config to include "trailing-whitespace" - Githubissues

commontk / CTK

A set of common support code for medical imaging, surgical navigation, and related purposes.

https://commontk.org

Apache License 2.0

827 stars 480 forks source link

Trim trailing whitespaces and update pre-commit config to include "trailing-whitespace" #1171

Closed jcfr closed 5 months ago

jcfr commented 5 months ago

Consistently avoiding trailing whitespaces is helpful to avoid "false differences" and ensure all diffing tools show the "real" changes by default.

By adding trailing-whitespace to the pre-commit configuration, it enables the following:

GitHub Action report: Suggested changes are reported in the error log associated with the pre-commit GitHub Action workflow.
Consolidated local pre-commit checks and updates by running:
```
pre-commit run --all-files
```

Adapted from:

https://github.com/Slicer/Slicer/pull/7332

Once this is integrated we will add the commit STYLE: Trim trailing whitespaces to .git-blame-ignore-revs^1 as it was one: