Skip to content

bpo-46341: remove duplicate paragraphs in asyncio documentation#30526

Closed
davem14 wants to merge 1 commit intopython:3.10from
davem14:patch-1
Closed

bpo-46341: remove duplicate paragraphs in asyncio documentation#30526
davem14 wants to merge 1 commit intopython:3.10from
davem14:patch-1

Conversation

@davem14
Copy link
Copy Markdown

@davem14 davem14 commented Jan 11, 2022

@davem14 davem14 requested review from 1st1 and asvetlov as code owners January 11, 2022 09:34
@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label Jan 11, 2022
@the-knights-who-say-ni
Copy link
Copy Markdown

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@davem14

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

Copy link
Copy Markdown
Contributor

@asvetlov asvetlov left a comment

Choose a reason for hiding this comment

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

The change is visible to the end user.
Please create an issue on https://bugs.python.org first.

Otherwise looks good.

@bedevere-bot
Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@davem14
Copy link
Copy Markdown
Author

davem14 commented Jan 11, 2022

I have made the requested changes; please review again

@bedevere-bot
Copy link
Copy Markdown

Thanks for making the requested changes!

@asvetlov: please review the changes made to this pull request.

@bedevere-bot bedevere-bot requested a review from asvetlov January 11, 2022 12:05
@AlexWaygood
Copy link
Copy Markdown
Member

AlexWaygood commented Jan 11, 2022

If you change the title of your PR to "bpo-46341: remove duplicate paragraphs in asyncio documentation", the bots will automatically link this PR to your BPO issue and turn the CI green :)

@davem14 davem14 changed the title remove duplicates bpo-46341: remove duplicate paragraphs in asyncio documentation Jan 11, 2022
@AlexWaygood
Copy link
Copy Markdown
Member

Btw, you've also used the 3.10 branch as the base for this PR — changes to CPython are nearly always made to the main branch (which is currently the development branch for as-yet-unreleased 3.11) first, and then backported to 3.10/3.9 if necessary :)

@davem14
Copy link
Copy Markdown
Author

davem14 commented Jan 11, 2022

So, what should I do now?

@AlexWaygood
Copy link
Copy Markdown
Member

Closing the PR and refiling against the main branch might be the easiest solution, I'm afraid :/ feel free to ping me if you do so, and I'll add the skip-news label to your new PR 🙂 you can obviously link your new PR to the same BPO ticket.

Also, as an aside, it would be great if you could sign the CLA, if you haven't already. It's not necessary for very small/trivial changes, but for legal reasons CPython can't accept significant contributions from people who haven't signed the CLA 🙂

@davem14 davem14 closed this Jan 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants