Make `ready_to_execute` channel more general

frequenz-floss / frequenz-dispatch-python

High-level interface to dispatch API

https://frequenz-floss.github.io/frequenz-dispatch-python/

MIT License

0 stars 3 forks source link

Make `ready_to_execute` channel more general #22

Closed Marenz closed 4 months ago

Marenz commented 6 months ago

Rename ready_to_execute channel to running_status_change reflecting it's broader purpose.
Small re-designs and changes about the interfaces in general
Add own Dispatch type with useful functions on top of the original base class

llucax commented 5 months ago

I see this was updated recently. Is it still a thing? Please let me know if you need more input from me.

Marenz commented 5 months ago

This is very much a thing, one that will be updated now that the dependencies are merged :)

Marenz commented 5 months ago

But apart from rebasing, the code is ready for review

llucax commented 5 months ago

@Marenz I updated the title to make it more suitable as a commit message. Can you please also update the PR description to be suitable as a commit message body? This will end up in the merge commit.

Marenz commented 4 months ago

@llucax anything missing for a merge? :)

llucax commented 4 months ago

Oh, sorry, I didn't see it was updated, will have a look now.