Skip to content

Don't pass Field metadata as keyword arg#675

Merged
lafrech merged 1 commit intodevfrom
fix_metadata_keyword_warning
Jun 4, 2021
Merged

Don't pass Field metadata as keyword arg#675
lafrech merged 1 commit intodevfrom
fix_metadata_keyword_warning

Conversation

@lafrech
Copy link
Copy Markdown
Member

@lafrech lafrech commented Jun 3, 2021

This is deprecated and generates warnings.

@lafrech lafrech force-pushed the fix_metadata_keyword_warning branch from 1f64e65 to f015664 Compare June 4, 2021 07:24
@lafrech
Copy link
Copy Markdown
Member Author

lafrech commented Jun 4, 2021

The only changes are in the tests, not in the source code (except for docstring examples).

This requires to bump marshmallow version to 3.10 for the tests but it is not a breaking change for the code itself. apispec still works with marshmallow >=3,< 3.10.

I don't think it is worth complicating things to try to also test on marshmallow 3.0. We can always drop marshmallow < 3.10 in next major and until then only test on 3.10+ and know it works on marshmallow 3.0.

@lafrech lafrech force-pushed the fix_metadata_keyword_warning branch from f015664 to 6d60a6c Compare June 4, 2021 20:39
@lafrech lafrech merged commit e95cdd4 into dev Jun 4, 2021
@lafrech lafrech deleted the fix_metadata_keyword_warning branch June 4, 2021 20:41
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.

1 participant