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

Add two spaces around equal sign in documentation #23719

Merged
merged 1 commit into from Dec 13, 2020

Conversation

sblondon
Copy link
Contributor

@sblondon sblondon commented Dec 9, 2020

So it fits to PEP8 coding style.

This PR is an aesthetic improvement, so I don't think it needs a bpo number nor a line in NEWS.

Fit to PEP8 coding style
@shihai1991
Copy link
Member

shihai1991 commented Dec 9, 2020

Thanks for your contribute. But almost PRs of code style will be rejected in here :)

@willingc
Copy link
Sponsor Contributor

willingc commented Dec 13, 2020

Thanks @shihai1991 for triaging this PR.

@sblondon, Thanks for your good intention with this PR. I encourage you in the future to refrain from submitting whitespace/style PRs unless the PR corrects a larger issue. Our volunteers' time is limited and the time cost for triaging and review don't outweigh the benefit of the change. Thanks in advance for respecting this in future PRs. ☀️

@willingc willingc merged commit da431f7 into python:master Dec 13, 2020
3 checks passed
@bedevere-bot
Copy link

bedevere-bot commented Dec 13, 2020

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

@sblondon
Copy link
Contributor Author

sblondon commented Dec 13, 2020

Thank for the merge ; ok for not sending PR about style in the future.

@sblondon sblondon deleted the add-missing-spaces branch Dec 13, 2020
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
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 issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants