Update line length from 80 to 130 characters

pixie16 / paass

Pixie Acquisition and Analysis Software Suite

https://pixie16.github.io/paassdoc/

GNU General Public License v3.0

10 stars 29 forks source link

Update line length from 80 to 130 characters #268

Closed spaulaus closed 7 years ago

spaulaus commented 7 years ago

I've had some discussions with my development team and they all advocate 130 character line limits. I was thinking that this might be good for this project since there are a number of lines that we've had to format in funny ways.

Are there any thoughts on this?

ksmith0 commented 7 years ago

More than 80 and it won't fit on a punch card anymore :-(

spaulaus commented 7 years ago

More than 80 and it won't fit on a punch card anymore :-(

I see that LANL has upped their security game.

spaulaus commented 7 years ago

@ksmith0 : I would like to reach a consensus on this. What's your vote?

ksmith0 commented 7 years ago

A consensus between you and me?

Well I used to just let the lines go as long as needed and that worked well for me. I see the advantage of finding a good place to break a line for readability and on modern computer 130 is not so long (although 120 is 50% more than 80 and that makes me happy for some reason). I vote for longner lines. It's silly to stick to a format from 1928 for a technology that has been obsolete for decades.

spaulaus commented 7 years ago

Not just a consensus between us but among the development group. Vote on my original post.

ksmith0 commented 7 years ago

Vote on my original post.

What does that mean?

spaulaus commented 7 years ago

What does that mean?

I meant we can add reactions to the original post to act as a voting system.

ksmith0 commented 7 years ago

Ahh, the mobile version doesn't show the reactions.