-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Comments
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. |
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. |
i don’t get it, all editors support autowrap, right? |
Stop it. It's not going to change. (And no, autowrap isn't always on and doesn't always do the right thing.) |
how can it do the wrong thing, though? |
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. |
I don't see what the problem is. Simply regard the unit of text as the paragraph. Line-oriented diffs aren't helpful, so I am with the editors on this. S
…Sent from my iPhone
On 12 Dec 2016, at 23:25, Brett Cannon ***@***.***> wrote:
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.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
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.
The text was updated successfully, but these errors were encountered: