Doc: Add link to interpreter tutorial by adorilson · Pull Request #16152 · python/cpython · GitHub
Skip to content

Doc: Add link to interpreter tutorial - #16152

Merged
matrixise merged 3 commits into
python:masterfrom
adorilson:patch-2
Sep 16, 2019
Merged

Doc: Add link to interpreter tutorial#16152
matrixise merged 3 commits into
python:masterfrom
adorilson:patch-2

Conversation

@adorilson

@adorilson adorilson commented Sep 14, 2019

Copy link
Copy Markdown
Contributor

Just a link to GNU Readline site.

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Sep 14, 2019
aeros
aeros suggested changes Sep 15, 2019
View reviewed changes

@aeros aeros left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR @adorilson.

+1 on adding the link, I just have a few minor suggestions.

  1. Grammar fix: "systems that support GNU Readline library" should be "systems that support the GNU Readline library".

  2. Use a direct inline link instead of the footer link. The footer links are typically used for links that are likely to be used multiple times within the page, but this one is only being used once. A direct link can be added using:

`GNU Readline <https://tiswww.case.edu/php/chet/readline/rltop.html>`_

(can be split across multiple lines)

  1. Adjust the number of characters per line to fit within the maximum reST 80 character width. Currently, line 37 is at 81 characters and line 40 is at 82 characters. I would suggest adjusting it after making the other adjustments, so that it doesn't have to be done multiple times.

@aeros

aeros commented Sep 15, 2019

Copy link
Copy Markdown
Contributor

@adorilson adorilson changed the title add link to GNU Readline site Doc: Add link to interpreter tutorial Sep 15, 2019
aeros
aeros approved these changes Sep 15, 2019
View reviewed changes

@aeros aeros left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, LGTM.

@matrixise matrixise self-assigned this Sep 16, 2019
matrixise
matrixise approved these changes Sep 16, 2019
View reviewed changes

@matrixise matrixise left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for your contribution

@matrixise
matrixise merged commit f18242b into python:master Sep 16, 2019
@miss-islington

miss-islington commented Sep 16, 2019

Copy link
Copy Markdown
Contributor

Thanks @adorilson for the PR, and @matrixise for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8.
🐍🍒⛏🤖

@miss-islington

miss-islington commented Sep 16, 2019

Copy link
Copy Markdown
Contributor

I'm having trouble backporting to 3.8. Reason: 'Error 110 while writing to socket. Connection timed out.'. Please retry by removing and re-adding the needs backport to 3.8 label.

@bedevere-bot

bedevere-bot commented Sep 16, 2019

Copy link
Copy Markdown

GH-16188 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 16, 2019
@adorilson @miss-islington
Doc: Add link of GNU Readline library to interpreter tutorial (python…
26a6202 
…GH-16152)

(cherry picked from commit f18242b)

Co-authored-by: Adorilson Bezerra <adorilson@gmail.com>
@miss-islington

miss-islington commented Sep 16, 2019

Copy link
Copy Markdown
Contributor

Thanks @adorilson for the PR, and @matrixise for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@bedevere-bot

bedevere-bot commented Sep 16, 2019

Copy link
Copy Markdown

matrixise pushed a commit that referenced this pull request Sep 16, 2019
@miss-islington @adorilson @matrixise
Doc: Add link of GNU Readline library to interpreter tutorial (GH-16152…
ad4ce9e 
…) (GH-16188)

(cherry picked from commit f18242b)

Co-authored-by: Adorilson Bezerra <adorilson@gmail.com>
matrixise pushed a commit that referenced this pull request Sep 16, 2019
@miss-islington @adorilson @matrixise
Doc: Add link of GNU Readline library to interpreter tutorial (GH-16152…
9dc381c 
…) (GH-16189)

(cherry picked from commit f18242b)

Co-authored-by: Adorilson Bezerra <adorilson@gmail.com>
@adorilson
adorilson deleted the patch-2 branch March 25, 2024 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matrixise matrixise matrixise approved these changes
+1 more reviewer
@aeros aeros aeros approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@matrixise matrixise

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@adorilson @aeros @miss-islington @bedevere-bot @matrixise @the-knights-who-say-ni