Skip to content
This repository has been archived by the owner on Jan 28, 2019. It is now read-only.

Settle on a convention for max line length #3

Open
sindrenm opened this issue Aug 7, 2015 · 5 comments
Open

Settle on a convention for max line length #3

sindrenm opened this issue Aug 7, 2015 · 5 comments

Comments

@sindrenm
Copy link
Contributor

sindrenm commented Aug 7, 2015

No description provided.

@Damian-Lopata
Copy link
Contributor

We should agree on this one and update playbook. I would recommend to have marker number set to 120. In that case the whole line of code is still visible on Retina 15inch screen. This is how it looks on my screen.
screen shot 2016-01-18 at 14 43 48

What do you think @ealmdahl @jeantuffier @sindrenm @changxiangzhong ?

@changxiangzhong
Copy link

120 looks good for me.

@jeantuffier
Copy link
Contributor

also fine for me

@ealmdahl
Copy link
Contributor

120 is ok with me too

@sindrenm
Copy link
Contributor Author

120 is totally fine by me, but am I the only one who thinks 4 spaces are a bit too much for indentation? I'll open another issue. (See #20.)

@sindrenm sindrenm mentioned this issue Jan 19, 2016
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@sindrenm @Damian-Lopata @jeantuffier @changxiangzhong @ealmdahl