Skip to content

Conversation

@bharel
Copy link
Contributor

@bharel bharel commented Feb 13, 2018

Alright.
Fixed, added tests, and added news.

https://bugs.python.org/issue32841

@1st1
Copy link
Member

1st1 commented Feb 13, 2018

@asvetlov Andrew please take a look.

@bharel
Copy link
Contributor Author

bharel commented Feb 13, 2018

@1st1 Hey mate, seems like the build failed because of a whitespace issue in the tests file. I have a fully enabled linter which doesn't really know what whitespace issue is it talking about.
May you lend me a hand here?

@bharel
Copy link
Contributor Author

bharel commented Feb 13, 2018

Alright. Seems like whitespace issues were fixed even though they weren't on any part of my code >.<

Copy link
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.

lgtm

@asvetlov asvetlov merged commit 5746510 into python:master Feb 14, 2018
@miss-islington
Copy link
Contributor

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

@bedevere-bot
Copy link

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

@bedevere-bot
Copy link

GH-5682 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 Feb 14, 2018
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 14, 2018
@bedevere-bot
Copy link

GH-5683 is a backport of this pull request to the 3.6 branch.

miss-islington added a commit that referenced this pull request Feb 14, 2018
asvetlov pushed a commit that referenced this pull request Feb 14, 2018
@bharel bharel deleted the bpo-32841 branch December 23, 2019 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants