-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat(Shard): shard-specific broadcastEval/fetchClientValues + shard Id util #4991
Conversation
0cc926b
to
ccbb40c
Compare
6205d8a
to
6aefbb3
Compare
6ff2d17
to
35d9e4d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code probably works but I have a few minor suggestions with the shardIdForGuildId
function.
7d2c438
to
e0d6da9
Compare
e0d6da9
to
4d1a66a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New edits look good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM
4d1a66a
to
8f6beec
Compare
Please describe the changes this PR makes and why it should be merged:
Adds the ability to call
broadcastEval
andfetchClientValues
from a shard to run on different target shard, instead of broadcasting to all and using anif
in the given script to filter by shard.Now you can target a shard directly:
We get back
[JetBrains Community, 1, 0]
, so we can see it correctly went to the target shard and found the guild there.Status
Semantic versioning classification: