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.
documenting.rst Outdated
@@ -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,::
Copy link
Member

@ezio-melotti ezio-melotti Oct 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Contributor Author

@jeff5 jeff5 Oct 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
@jeff5 jeff5 deleted the 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