Skip to content

Adding basic events to celery Batches#82

Closed
Anghille wants to merge 2 commits intoclokep:mainfrom
Anghille:main
Closed

Adding basic events to celery Batches#82
Anghille wants to merge 2 commits intoclokep:mainfrom
Anghille:main

Conversation

@Anghille
Copy link

@Anghille Anghille commented Oct 2, 2023

Following preceding works (not updated since), I do this PR to add the following modifications

  • Add signals.task_received to tests
  • Add events task-received, task-succeded and task-started
  • Format to follow black /flake8 formatting

I intend (with your help if possible) to implement at least task-failed and task-retried events as well.

* Add events task-received, task-succeded and task-started
* Format to follow black formating
@clokep
Copy link
Owner

clokep commented Oct 2, 2023

Following preceding works (not updated since)

I think this is based on #34?

@clokep clokep closed this Apr 4, 2025
@clokep
Copy link
Owner

clokep commented Apr 4, 2025

I think this needs a rework -- the tests don't seem to quite make sense to the code changes. If you're still interested in working on it, let me know and we can reopen.

@Anghille
Copy link
Author

Anghille commented Apr 4, 2025

I might take a look at some point, but I managed to do a workaround using an uglier solution for now (that is outside of this repo scope). It won't stand longterm but works for now. Thanks :)

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