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

Clean up trailing white space #66

Open
goneall opened this issue Jul 1, 2020 · 1 comment
Open

Clean up trailing white space #66

goneall opened this issue Jul 1, 2020 · 1 comment
Assignees

Comments

@goneall
Copy link
Member

goneall commented Jul 1, 2020

There is a significant amount of trailing white space in the java files.

We can use git-stripspace as a consistent method of stripping white space.

This will need to be applied to all .java files in the repo. After which, we can include git-stripspace as a git hook before committing changes to keep things consistent.

@goneall goneall self-assigned this Nov 12, 2020
@CAM-Gerlach
Copy link

@goneall Quick suggestion: The pre-commit cross platform hooks framework is a fantastic tool for automating this and many other such things; it supports installing, running and updating hundreds of hooks in isolated environments across a dozen or so supported languages (including built in hooks for common things like whitespace), including many that support automated dependency installation and several (including the most popular) that don't even need the language runtime to be installed, which greatly eases the workflow for developers and contributors, as initial setup only requires one command and everything after that is automated, plus it ensures every developer is using consistent tool versions to avoid commits failing on some machines but not others. It can also be run in CI, either in any CI or via the pre-commit.ci service.

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

No branches or pull requests

2 participants