Skip to content

Upgrade to Celery 2.5.3#627

Closed
rlr wants to merge 2 commits intomasterfrom
celery-253
Closed

Upgrade to Celery 2.5.3#627
rlr wants to merge 2 commits intomasterfrom
celery-253

Conversation

@rlr
Copy link
Contributor

@rlr rlr commented Jun 1, 2012

Not much of interest here. Some fixes for deprecation warnings and a bunch of dependency updates.

I am going to try this on stage now.

r?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What requires ordereddict? Python has its own in the collections module. I wonder if they differ somehow.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From what I could tell, it is new in python 2.7?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, whoops--you're right. It's new in 2.7. Nevermind!

@willkg
Copy link
Member

willkg commented Jun 4, 2012

Awesome that you fixed django-tidings. Do we have a bug for updating that?

@rlr
Copy link
Contributor Author

rlr commented Jun 4, 2012

I was planning on including the django-tidings update right here.

@willkg
Copy link
Member

willkg commented Jun 4, 2012

I'm getting one django-tidings warning regarding celery.decorators. Other than that, everything looks good to me. r+

@willkg
Copy link
Member

willkg commented Jun 4, 2012

Landed in 55d2999

@willkg willkg closed this Jun 4, 2012
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.

2 participants