You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by GitBox <gi...@apache.org> on 2018/11/29 04:29:52 UTC
[GitHub] jaydoane edited a comment on issue #1769: Add Credo to Elixir test
suite
jaydoane edited a comment on issue #1769: Add Credo to Elixir test suite
URL: https://github.com/apache/couchdb/pull/1769#issuecomment-442692466
@wohali I much prefer 90 to 120. And you are right about numerous variables factoring into how many characters are comfortable to squeeze on a screen and read easily over long time periods.
Hmm, I'm not sure I find it more readable with all the additional vertical space it uses in some places [going from 120 to 90](https://github.com/apache/couchdb/compare/master...cloudant:format-elixir-line-length-90?expand=1).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services