Skip to content

feat: broadcast message to all workers - #47

Draft
Akryum wants to merge 2 commits into
tinylibs:mainfrom
Akryum:feat/message-all
Draft

feat: broadcast message to all workers#47
Akryum wants to merge 2 commits into
tinylibs:mainfrom
Akryum:feat/message-all

Conversation

@Akryum

@Akryum Akryum commented Feb 17, 2023 •
edited
Loading

Copy link
Copy Markdown

related to #25
@Akryum

Akryum commented Feb 17, 2023

Copy link
Copy Markdown
Author

histoire-dev/histoire@0cbf351
Aslemammad
Aslemammad reviewed Feb 17, 2023
View reviewed changes

@Aslemammad Aslemammad left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. We just need a test case and a small documentation.

By the way, what if instead of broadcastMessage, we do pool.runInAll or something like that.
@Aslemammad

Aslemammad commented Mar 7, 2023

Copy link
Copy Markdown
Member

@Akryum Any update on this?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@Akryum @Aslemammad