Require deriving explicitly ActionBuilder
or ScorerBuilder
#62
Labels
enhancement
New feature or request
good first issue
Good for newcomers
help wanted
Extra attention is needed
I think it would be helpful to require users to derive explicitly
ActionBuilder
andScorerBuilder
(and to change all theimpl ActionBuilder
intoActionBuilder
, so that users need to provide explicitly aActionBuilder
).ScorerBuilder
was, and how it was related to me just creating a struct withComponent
andClone
.I think it would be much clearer for users to have their scorers-builders/action-builders with explicit
#[derive(ScorerBuilder)]
This is the exact same argument as: bevyengine/bevy#1843
In theory, it's more flexible to accept any
std::fmt::Debug + Sync + Send
as a ScorerBuilder (by allowing pre-existing types to beScorerBuilders
, but in practice it just makes it less clear, and i believe 99% of users won't use a pre-existing type as a scorer builder)ScorerBuilder
to have aLabel
.By forcing all ScorerBuilders to have the explicit type, users could easily add the label via
The text was updated successfully, but these errors were encountered: