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

fix(mm): Replace deprecated Marshmallow Field parameters #351

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stevenpackardblp
Copy link

In Marshmallow 3.13.0, the default and missing parameters of the
Field object were deprecated and replaced with dump_default and
load_default respectively. As these are new parameters added in
version 3.13.0, the minimum version requirement is bumped accordingly.

fixes: #328

@stevenpackardblp
Copy link
Author

This would be nice to clean up the warning spews!

@camin-mccluskey
Copy link

In case anyone is seeing hundreds of errors, is waiting on this fix to be merged (and uses pytest) adding the following to a pytest.ini file will suppress these warnings:

[pytest]
filterwarnings =
    ignore::DeprecationWarning:marshmallow*
    ignore::DeprecationWarning:dataclasses_json*

@andret13pinto
Copy link

I would love for this fix to be merged!

@george-zubrienko
Copy link
Collaborator

@stevenpackardblp please update your PR and we can merge ;)

@george-zubrienko
Copy link
Collaborator

@stevenpackardblp another ping :) if no reply in a week I'll take over this PR and re-create it

@george-zubrienko george-zubrienko self-assigned this Jul 20, 2023
@stevenpackardblp
Copy link
Author

@stevenpackardblp another ping :) if no reply in a week I'll take over this PR and re-create it

Let me refresh my workspace this week and see if I can get this rebased. Apologies for missing the first ping.

@russwinch
Copy link

it would be great if this could be merged 🙏

@rbreslavsky
Copy link

Agree, would be great if this could be addressed!

@stevenpackardblp
Copy link
Author

@george-zubrienko big apologies for the delay, I just rebased it.

In Marshmallow 3.13.0, the `default` and `missing` parameters of the
`Field` object were deprecated and replaced with `dump_default` and
`load_default` respectively.

fixes: lidatong#328
@stevenpackardblp
Copy link
Author

rebased again.

@nomantis
Copy link

nomantis commented Aug 1, 2024

It would be great to get this merged, so the deprecation warnings can finally be addressed 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Marshmallow warnings
7 participants