Skip to content

Do away with the 79 char limit #157

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
flying-sheep opened this issue Dec 12, 2016 · 7 comments
Closed

Do away with the 79 char limit #157

flying-sheep opened this issue Dec 12, 2016 · 7 comments

Comments

@flying-sheep
Copy link
Contributor

It makes sense for code to have a line length limit, but not for prose:

When editing a document with manual breaks, lines will either be constantly uneven or you will have to manually reflow whole paragraphs all the time, breaking diffs.

I propose to do one sentence per line, and break after commas in long composite sentences. This will be nice for diffs, and result in no useless manual reflowing.

@warsaw
Copy link
Member

warsaw commented Dec 12, 2016

I disagree. PEPs are edited by the same people and in the same editing environments as code, so having such a limit makes it more compatible across disparate editing environments. E.g. lines which are too long can be autowrapped or syntax highlighted in a warning color.

TBH, it also always bothers me when I see Python documentation source files with too long lines.

@brettcannon
Copy link
Member

I agree with Barry; I write PEPs in code editors for their reST support and so the lack of line breaks doesn't work. Plus I personally read plenty of PEPs in raw text form.

I think two PEP editors not in favour of the idea is enough to close this.

@flying-sheep
Copy link
Contributor Author

i don’t get it, all editors support autowrap, right?

@gvanrossum
Copy link
Member

Stop it. It's not going to change. (And no, autowrap isn't always on and doesn't always do the right thing.)

@flying-sheep
Copy link
Contributor Author

how can it do the wrong thing, though?

@brettcannon
Copy link
Member

If you wrap hard at the column limit with hyphens to get a justified look that can break words at bad points.

But it really doesn't matter as we are not changing this requirement/practice for other reasons as well (e.g. not requiring everyone to turn on auto-wrap), so please consider this issue closed.

@holdenweb
Copy link
Member

holdenweb commented Dec 20, 2016 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants