bpo-36620 [First-time contributor] Update itertools.rst add optional args to function definitions. by yingw787 · Pull Request #13210 · python/cpython · GitHub
Skip to content

bpo-36620 [First-time contributor] Update itertools.rst add optional args to function definitions. - #13210

Closed
yingw787 wants to merge 6 commits into
python:masterfrom
yingw787:master
Closed

yingw787 wants to merge 6 commits into
python:masterfrom
yingw787:master

Conversation

@yingw787

@yingw787 yingw787 commented May 8, 2019

Copy link
Copy Markdown
Contributor

@the-knights-who-say-ni

Copy link
Copy Markdown

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels May 8, 2019
@yingw787

yingw787 commented May 8, 2019

Copy link
Copy Markdown
Contributor Author

@lisroach pls review thanks :)

@@ -0,0 +1 @@
Update itertools documentation with optional args registration. No newline at end of file

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.

Can you actually remove the NEWS entry? For small documentation fixes an NEWS entry isn't really needed, and can end up cluttering up the news at the end of the day.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@lisroach should be removed! I thought it was required because the CI checks had a large red X on not having a NEWS entry filled out, then the category disappeared after I filled it out (not sure if it's because I filled it out and it completed or if the CI checks deleted the category after PR instantiation).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay I get it now; the .rst file must be present but the entry is "skip news" for the bot to skip news. Updated.

@bedevere-bot

Copy link
Copy Markdown

@rhettinger rhettinger self-assigned this May 9, 2019
@rhettinger rhettinger closed this May 10, 2019
jcfr added a commit to python-cmake-buildsystem/cpython that referenced this pull request Jan 22, 2020
…om VS2008 to VS2010."

This commit is a partial backport of python/cpython@401f9f3. It was
originally designed to work with python-cmake-buildsystem.

The following modules have NOT been backported:

* Tools/msi
* Tools/buildbot
* PCBuild
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changes docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants