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

stdlib: Remove duplicated Pickler.reducer_override() #3772

Merged
merged 1 commit into from
Feb 22, 2020

Conversation

moreati
Copy link
Contributor

@moreati moreati commented Feb 22, 2020

The declaration was present twice. One from #3358 with the bulk Python
3.8 support, the other from #3636 with Pickle protocol 5 support.

@srittau
Copy link
Collaborator

srittau commented Feb 22, 2020

Thanks! Please remove pickle.Pickler.reducer_override-redefinition from tests/stubtest_whitelists/py38.txt to fix the CI failure.

The declaration was present twice. One from python#3358 with the bulk Python
3.8 support, the other from python#3636 with Pickle protocol 5 support.
@moreati moreati force-pushed the duplicated-reducer_override branch from 59b6b6f to b45af5e Compare February 22, 2020 16:39
@srittau
Copy link
Collaborator

srittau commented Feb 22, 2020

That CI is still failing is unrelated to this PR. I will merge it, once Build #8898 passes.

@srittau srittau merged commit 84c6e67 into python:master Feb 22, 2020
@moreati moreati deleted the duplicated-reducer_override branch February 22, 2020 21:54
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.

2 participants