-
Notifications
You must be signed in to change notification settings - Fork 26
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: changed marshmallow-annotation version, temp solution #81
Conversation
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need to bump the version of common as well?
Would this be a breaking change for any service on marshmallow < 3.0? |
Slightly related to this, I have created RFC to introduce Python dataclasses instead of using attrs. |
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
56c59c1
Summary of Changes
Using git+https://www.github.com/hilearn/marshmallow-annotations.git@a7a2dc96932430369bdef36555082df990ed9bef#egg=marshmallow-annotations as marshmallow-annotations version while waiting for dep to be maintained and compatible with marshmallow >3.0.
CheckList
Make sure you have checked all steps below to ensure a timely review.