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

removed unused error argument #1209

Merged
merged 1 commit into from
May 6, 2019

Conversation

rooterkyberian
Copy link
Contributor

seems like some kind of leftover

@lafrech
Copy link
Member

lafrech commented May 6, 2019

No idea what that is.

It is there in 2.x as well and it seems just as useless, so it must be a pretty old leftover, unrelated to the error structure rework we did in MA3. (I'm not saying we should change this in 2.x. I just looked there to try and understand.)

@sloria
Copy link
Member

sloria commented May 6, 2019

Ah good catch! I think this is a holdover from marshmallow 1.

@sloria sloria merged commit 5ec6df6 into marshmallow-code:dev May 6, 2019
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.

None yet

3 participants