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

Copy link
Copy Markdown

related to #25

@Akryum

Akryum commented Feb 17, 2023

Copy link
Copy Markdown
Author

@Aslemammad Aslemammad left a comment

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants