exercism / exercism/org-wide-files

consider adding org-wide workflow to lint whitespace

Open
#213 6 comments 0 reactions 0 assignees View on GitHub
Dominant language
Shell
Stars
7
Forks
8
PR merge metrics
No merged PRs in 30d

Description

Some arguments in https://github.com/exercism/org-wide-files/pull/204#pullrequestreview-950940243.

And from e.g. https://www.kernel.org/doc/html/v4.10/process/coding-style.html
> Do not leave trailing whitespace at the ends of lines. Some editors with smart indentation will insert whitespace at the beginning of new lines as appropriate, so you can start typing the next line of code right away. However, some such editors do not remove the whitespace if you end up not putting a line of code there, such as if you leave a blank line. As a result, you end up with lines containing trailing whitespace.
>
>Git will warn you about patches that introduce trailing whitespace, and can optionally strip the trailing whitespace for you; however, if applying a series of patches, this may make later patches in the series fail by changing their context lines.

If we wanted to add whitespace linting, we could do it in `configlet lint`. But then we'd only lint whitespace in repos where `configlet lint` runs. So it seems an org-wide workflow is better.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.