Skip to content

Compatibility with marshmallow 4 #259

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 5 commits into from
May 4, 2025
Merged

Conversation

decaz
Copy link
Contributor

@decaz decaz commented Apr 17, 2025

Fixes #255, fixes #260.

@decaz
Copy link
Contributor Author

decaz commented Apr 17, 2025

@jmcarp @sloria take a look please.

@sloria
Copy link
Collaborator

sloria commented May 4, 2025

Thanks! Looks correct to me so going to go ahead and merge. unfortunately i don't have the time rn to update the CI setup (should probably migrate to GHA) and can't release until that happens. neither i nor jmcarp are currently using this project in our day jobs so don't have a lot of time to invest atm. if you're looking for a marshmallow-based flask API library, i'd recommend taking a look at https://github.com/marshmallow-code/flask-smorest, which is actively maintained by a marshmallow co-maintainer.

that said, if someone sends a PR with the GHA migration i would review and merge it

@sloria sloria merged commit 2e6717d into jmcarp:master May 4, 2025
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.

Incompatibility with marshmallow 4 DeprecationWarning about __version__ attributes
3 participants