Skip to content

bpo-40045: Make "dunder" method documentation easier to locate#19153

Merged
nanjekyejoannah merged 2 commits into
masterfrom
unknown repository
Mar 27, 2020
Merged

bpo-40045: Make "dunder" method documentation easier to locate#19153
nanjekyejoannah merged 2 commits into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Mar 25, 2020

Copy link
Copy Markdown

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Mar 25, 2020
@ghost ghost changed the title bpo-40045 bpo-40045: Make "dunder" method documentation easier to locate Mar 25, 2020

@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 @JavadMokhtari; LGTM.

@nanjekyejoannah nanjekyejoannah 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.

Since there was consensus from bpo here : https://bugs.python.org/issue40045, this LGTM. Thanks for your contribution @JavadMokhtari .

Some minor consistent line length changes
@aeros

aeros commented Mar 26, 2020

Copy link
Copy Markdown
Contributor

IMO, this should be backported to 3.8 and 3.7.

@nanjekyejoannah nanjekyejoannah 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.

Let us give @rhettinger time to look. If he doesn't, I will merge this tomorrow.

@nanjekyejoannah nanjekyejoannah merged commit 5f9c131 into python:master Mar 27, 2020
@miss-islington

Copy link
Copy Markdown
Contributor

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

@bedevere-bot

Copy link
Copy Markdown

@nanjekyejoannah: Please replace # with GH- in the commit message next time. Thanks!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 27, 2020
…nGH-19153)

* issue 40045

* Update lexical_analysis.rst

Make "dunder" method documentation easier(pythonGH-19153)

Co-authored-by: Joannah Nanjekye <33177550+nanjekyejoannah@users.noreply.github.com>
(cherry picked from commit 5f9c131)

Co-authored-by: Javad Mokhtari <javadmokhtari@outlook.com>
@bedevere-bot

Copy link
Copy Markdown

GH-19198 is a backport of this pull request to the 3.8 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 27, 2020
…nGH-19153)

* issue 40045

* Update lexical_analysis.rst

Make "dunder" method documentation easier(pythonGH-19153)

Co-authored-by: Joannah Nanjekye <33177550+nanjekyejoannah@users.noreply.github.com>
(cherry picked from commit 5f9c131)

Co-authored-by: Javad Mokhtari <javadmokhtari@outlook.com>
@bedevere-bot

Copy link
Copy Markdown

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

@aeros

aeros commented Mar 27, 2020

Copy link
Copy Markdown
Contributor

Congrats on your first merged CPython PR, @JavadMokhtari! :-)

nanjekyejoannah pushed a commit that referenced this pull request Mar 27, 2020
) (GH-19198)

* issue 40045

* Update lexical_analysis.rst

Make "dunder" method documentation easier(GH-19153)

Co-authored-by: Joannah Nanjekye <33177550+nanjekyejoannah@users.noreply.github.com>
(cherry picked from commit 5f9c131)

Co-authored-by: Javad Mokhtari <javadmokhtari@outlook.com>

Co-authored-by: Javad Mokhtari <javadmokhtari@outlook.com>
nanjekyejoannah pushed a commit that referenced this pull request Mar 27, 2020
) (GH-19199)

* issue 40045

* Update lexical_analysis.rst

Make "dunder" method documentation easier(GH-19153)

Co-authored-by: Joannah Nanjekye <33177550+nanjekyejoannah@users.noreply.github.com>
(cherry picked from commit 5f9c131)

Co-authored-by: Javad Mokhtari <javadmokhtari@outlook.com>

Co-authored-by: Javad Mokhtari <javadmokhtari@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants