cothority
cothority copied to clipboard
Add line-length check to Travis
Respecting the line-length in the code (80 characters) should be done in a travis-check.
Also, could this be something like 110-130 ? 80 is from the old days .. or is it better for the emacs guys ?
Also, could this be something like 110-130 ? 80 is from the old days .. or is it better for the emacs guys ?
At the end this is matter of taste, but I find 80 chars to have several (but arguable) benefits:
- Makes the code easier to read without needing to move your eyes horizontally
- Helps keeping a code simple and better reveals code smells (if your function needs 3 lines for its arguments declaration then maybe it should be refactored)
- Allows to keep multiple files opened side-by-side on the same window
- Is a well established standard
but yes, 80 chars feels sometimes frustrating, Python eventually took me on this one...
I think the limit originally was Bryan's (strong) suggestion ;)
I would never call for a revolution, of course...
What about adding a code-quality check like https://www.codacy.com ?