You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The signal library is great for modules, but sometimes I simply need 2 imperative scripts under the same context to talk to each other, and bindables fit that use case just fine (and avoid having to add the Signal utility as a dependancy for Net).
An example use case would be an egg in an egg hunt, where each egg has it's own behaviour but they all must communicate to the manager coordinating the eggs that it has been collected (on this case the manager is imperative as it doesn't have enough functions to justify making a module, thus signals wouldn't make a whole lot of sense).
Alternatively a 'Name' method could be added to the signal utility that just creates and/or returns a signal from a dictionary, but 'Net' seemed more fitting imo.
The signal library is great for modules, but sometimes I simply need 2 imperative scripts under the same context to talk to each other, and bindables fit that use case just fine (and avoid having to add the Signal utility as a dependency for Net).
I agree with you regarding the dependency footprint, though the reason we often use the Signal utility is for defensive programming, and to maintain a consistent execution structure across our projects.
Mixing C2C (context-to-context) communication patterns directly within Net could lead to an inconsistent API that's a bit harder to maintain.
I'd say best course of action is keeping this as is for now, though I think a PR for a BindableEvent wrapper that sits atop of the Net library for your specific needs would be great!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The signal library is great for modules, but sometimes I simply need 2 imperative scripts under the same context to talk to each other, and bindables fit that use case just fine (and avoid having to add the Signal utility as a dependancy for Net).
An example use case would be an egg in an egg hunt, where each egg has it's own behaviour but they all must communicate to the manager coordinating the eggs that it has been collected (on this case the manager is imperative as it doesn't have enough functions to justify making a module, thus signals wouldn't make a whole lot of sense).
Alternatively a 'Name' method could be added to the signal utility that just creates and/or returns a signal from a dictionary, but 'Net' seemed more fitting imo.