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

STY: remove trailing whitespace #57

Closed
wants to merge 1 commit into from

Conversation

klauer
Copy link
Contributor

@klauer klauer commented May 27, 2020

Using pre-commit to help us out:

.pre-commit-config.yaml

# See https://pre-commit.com for more information
# See https://pre-commit.com/hooks.html for more hooks
repos:
-   repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v2.5.0
    hooks:
    -   id: no-commit-to-branch
    -   id: trailing-whitespace
$ pip install pre-commit
$ pre-commit install
$ pre-commit run --all-files

This can help even if your editor doesn't support fixing trailing whitespace, for example, by automatically running the hooks defined in the config file on each commit.

Copy link
Contributor

@prjemian prjemian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually I am completely hands-off with modifications to the LICENSE file, even cosmetic changes such as re-flowing paragraphs. These changes will not be noticed by reasonable people.

tldr: OK

@prjemian
Copy link
Contributor

If you have other changes, tag me again for review.

@klauer klauer closed this Jun 30, 2020
@klauer klauer deleted the trailing_whitespace branch March 18, 2021 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants