Skip to content
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

Limit line length to 80 & clean-up nearby formatting. #282

Merged
merged 2 commits into from Oct 13, 2017

Conversation

@jeff5
Copy link
Contributor

@jeff5 jeff5 commented Oct 12, 2017

Primary aim is to conform to the 80 character rule (where possible). At the same time I've cleaned up other formatting faults in the same area as the long lines.

I don't personally like to re-flow paragraphs further than end of the affected sentence when I'm writing, so I mostly haven't. It makes it easier to see what changed.

Wide-ranging but essentially trivial adjustment of line length. Also
fixes top-level indenting in gdb.rst particularly.
@@ -456,7 +457,7 @@ the extension mechanisms of reST, and Sphinx makes heavy use of it.

Basically, a directive consists of a name, arguments, options and content. (Keep
this terminology in mind, it is used in the next chapter describing custom
directives.) Looking at this example, ::
directives.) Looking at this example,::

This comment has been minimized.

@ezio-melotti

ezio-melotti Oct 13, 2017
Member

Isn't the space necessary here?
You could remove the comma and use example::, but otherwise I would leave it as is.

This comment has been minimized.

@jeff5

jeff5 Oct 13, 2017
Author Contributor

That's a bit of Eclipse formatting (good for some things), not intentional. I see that in the original, the comma shows and the colon is invisible. I'll replace with the long form.

@ezio-melotti ezio-melotti merged commit 8c30364 into python:master Oct 13, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jeff5 jeff5 deleted the jeff5:cp-linelength branch Oct 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants