amethyst / shred

Shared resource dispatcher
Apache License 2.0
233 stars 66 forks source link

Added DispatcherBuilder::contains #211

Closed schell closed 2 years ago

schell commented 2 years ago

Adds a contains method to DispatcherBuilder that allows the caller to determine if a system with the given name has already been added.

This supports https://github.com/amethyst/specs/issues/736.

schell commented 2 years ago

@torkleyy this one should be a really easy review :)

schell commented 2 years ago

Any luck with this? Is there anything else I should do to get this merged?

bors[bot] commented 2 years ago

Build failed:

torkleyy commented 2 years ago

The error wasn't introduced by this PR so I'll just merge this manually.

torkleyy commented 2 years ago

Actually, let me try and fix the issue and then merge it :)

torkleyy commented 2 years ago

bors r+

bors[bot] commented 2 years ago

Build succeeded:

schell commented 2 years ago

No worries @torkleyy, thank you!