Skip to content
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

Update to support celery 3 and celery 4 option parsing #60

Merged
merged 2 commits into from
Dec 24, 2016

Conversation

belasin
Copy link
Contributor

@belasin belasin commented Nov 28, 2016

Tweak to the config parser to support both celery 3 and 4

@coveralls
Copy link

coveralls commented Nov 28, 2016

Coverage Status

Changes Unknown when pulling 4a4342c on jpunwin:przemal/master into ** on sontek:master**.

@belasin belasin changed the title Przemal/master Use new Celery 4 argument parser (argparse instead of optparse) Nov 28, 2016
@belasin
Copy link
Contributor Author

belasin commented Nov 28, 2016

Not sure if there is a way to merge this with #59

@belasin belasin changed the title Use new Celery 4 argument parser (argparse instead of optparse) Update to support celery 3 and celery 4 option parsing in cmdline Nov 28, 2016
@belasin belasin changed the title Update to support celery 3 and celery 4 option parsing in cmdline Update to support celery 3 and celery 4 option parsing Nov 28, 2016
@alexsv
Copy link

alexsv commented Nov 30, 2016

+1 for this change

@przemal
Copy link

przemal commented Nov 30, 2016

@jpunwin Closed #59, so this can be merged instead

@alexsv
Copy link

alexsv commented Dec 16, 2016

Guys, is it going to be merged?

@belasin
Copy link
Contributor Author

belasin commented Dec 20, 2016

Shrug still waiting on approval

@sontek sontek merged commit 93bf846 into sontek:master Dec 24, 2016
@sontek
Copy link
Owner

sontek commented Dec 24, 2016

This is released on pypi as pyramid_celery 3.0.0 with some other fixes that I had in the queue! Thanks for the contribution

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.

5 participants