search

coderofsalvation / powscript

transpiler written in bash: painless shellscript, indentbased, coffee for the shell with hipster-sparkles v1 BETA LANDED 🎉🎉🎉🎉 thanks fcard!
http://powscript.isvery.ninja/
Other
159 stars 13 forks source link

Remove trailing whitespace #34

Closed fcard closed 7 years ago

fcard commented 7 years ago

Not allowing trailing whitespace is a habit I got from other projects... And because of those projects I have my editor configured to show me an annoying red highlight every time there is any left over whitespace. I imagine that to be true of many other people as well. (and since powscript is indent based I think the ideal is to have our editors detect trailing whitespace anyway)

This can probably be automated with a git hook as well. Maybe we should have a .tools/hooks directory with a few useful git hooks and a .tools/install_hooks script to install them.

fcard commented 7 years ago

This is a fairly simple non-disruptive change so I was thinking of just merging it, but I want to know if you're down for making this an official policy, @coderofsalvation (I don't mind reinforcing it by myself)

(also sorry if I am distracting you from your work, feel free to take your time)

coderofsalvation commented 7 years ago

good idea! I like 'householding' githooks as well.