Skip to content

bpo-33253: Fix xxsubtype.bench to accept correct string signature #6439

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

Merged
merged 1 commit into from
Apr 10, 2018

Conversation

scarletcafe
Copy link
Contributor

@scarletcafe scarletcafe commented Apr 9, 2018

This fixes the issue as explained on the tracker where xxsubtype.bench can never run without raising an exception due to the old Python 2 string signature still being in use.

Old behavior:
image

New behavior:
image

https://bugs.python.org/issue33253

@the-knights-who-say-ni
Copy link

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately our records indicate you have not signed the CLA. For legal reasons we need you to sign this before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

Thanks again to your contribution and we look forward to looking at it!

@serhiy-storchaka serhiy-storchaka merged commit f178028 into python:master Apr 10, 2018
@miss-islington
Copy link
Contributor

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

@bedevere-bot
Copy link

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

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

@serhiy-storchaka
Copy link
Member

Thank you for your contribution @Gorialis!

Signing the CLA is not required for such trivial change. But it would be better to sign it for the case if you will contribute less trivial changes in future.

miss-islington added a commit that referenced this pull request Apr 10, 2018
miss-islington added a commit that referenced this pull request Apr 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants