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
Fixed #25764 -- Added support for serialization of enum.Enum in migrations. #5672
Conversation
@@ -642,6 +642,8 @@ Django can serialize the following: | |||
|
|||
- ``int``, ``long``, ``float``, ``bool``, ``str``, ``unicode``, ``bytes``, ``None`` | |||
- ``list``, ``set``, ``tuple``, ``dict`` | |||
- Instance of ``enum.Enum`` (either native for Python 3.4+, or ``Enum`` from enum34_ |
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.
``enum.Enum`` instances
Please add a mention in the 1.10 release notes and squash commits using the commit message from the pull request title. Thanks! |
@timgraham Could you have a look one more time before I squash the commits? |
23887db
to
6e21ada
Compare
@@ -138,6 +138,7 @@ dependencies: | |||
|
|||
* bcrypt_ | |||
* docutils_ | |||
* enum34_ 2.7-3.3 |
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.
2.7+ below refers to the version of jinja2, we can change this to "(Python 2 only)"
6e21ada
to
610d4a1
Compare
@timgraham fixed |
610d4a1
to
ff9c324
Compare
(removed remaining reference) |
Some minor edits: http://dpaste.com/3AXZQX0 Upon installing enum34 on Python 2, I noticed the test doesn't pass there. |
ff9c324
to
8ce355b
Compare
Is it possible to use a similar fix?
|
Yeah, trying to apply it. I think I should serialize the enum member value as well, working on it |
b11bab4
to
2422d22
Compare
@timgraham it seems that jenkins is hanging, I'll try again |
2422d22
to
916726c
Compare
Thanks Tim Graham for the review.
916726c
to
f59fe22
Compare
@timgraham Jenkins seems randomly hang or fail for Windows |
It's under a lot of load now. Try to avoid pushing your branch a lot (e.g. pushing a new commit, then immediately pushing a squashed one) since this will trigger two builds. |
@timgraham Is there a way to trigger tests without pushing? |
See https://code.djangoproject.com/wiki/Jenkins but there's no need to retrigger just for random failures. |
merged in 998894e, thanks! |
Ticket https://code.djangoproject.com/ticket/25764