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.

Sign up for GitHub

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

Jump to bottom

bpo-34204: Make the shelve library use Pickle's default protocol version instead of defaulting to version 3 #22751

Closed
wants to merge 1 commit into from
Closed

bpo-34204: Make the shelve library use Pickle's default protocol version instead of defaulting to version 3 #22751

wants to merge 1 commit into from

Conversation

marco-c
Copy link

@marco-c marco-c commented Oct 18, 2020 •
edited
Loading

@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 this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

CLA Missing

Our records indicate the following people have not signed the CLA:

@marco-c

For legal reasons we need all the people listed to sign the CLA before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

If you have recently signed the CLA, please wait at least one business day
before our records are updated.

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

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

@marco-c marco-c changed the title bpo-42071: Make the shelve library use Pickle's default protocol version instead of defaulting to version 3 bpo-34204: Make the shelve library use Pickle's default protocol version instead of defaulting to version 3 Oct 27, 2020
@marco-c marco-c mentioned this pull request Oct 28, 2020
Merged
@marco-c
Copy link
Author

marco-c commented Oct 28, 2020

@vstinner r?

@ZackerySpytz
Copy link
Contributor

@marco-c Please close this PR -- it is a duplicate. If you want, I can add your name to Doc/whatsnew/3.10.rst in my PR.

@csabella
Copy link
Contributor

Closing this as the original PR has been merged.

@csabella csabella closed this Oct 29, 2020
@serhiy-storchaka serhiy-storchaka mentioned this pull request Apr 10, 2022
Closed
@marco-c marco-c mannequin mentioned this pull request Apr 10, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
awaiting review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@marco-c @the-knights-who-say-ni @ZackerySpytz @csabella @bedevere-bot