-
Notifications
You must be signed in to change notification settings - Fork 14
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
setup: bump Celery version #24
base: master
Are you sure you want to change the base?
Conversation
4c1dda5
to
2a4e209
Compare
You'll want to make this generate a major version, otherwise you're going to break the build! |
As for this build not passing, see: inveniosoftware/troubleshooting#8. |
2a4e209
to
bb39281
Compare
Let's see if it works now, although even applying the changes in inveniosoftware/invenio-indexer@e044428 I am getting the error locally.
@david-caro is there a way to mark it in the commit message? |
Yes, you can even do it from the ui on github when you hit the merge button, you can add to it (at the end):
That will tag it as major change. |
Depends on celery/vine#18 |
The issue with Celery was fixed in celery/celery#4399. Once a new release of Celery 4 is done, this can unblocked. |
No description provided.