Do we need both run_if and in_set?

[torsteingrindvik]

In my code I have lots of systems which end in

.run_if(in_state(STATE))
.in_set(SET),

and in_set just feels like another run_if condition.

In the spirit of reducing API surface, would it be better to be able to just

.run_if(in_state(STATE))
.run_if(in_set(SET)),

?

[alice-i-cecile]

System sets are a more fundamental primitive than run conditions: they can be nested, ordered, configured and so on.

In code like your first example, I tend to try and attach the run condition to the set itself.

[torsteingrindvik]

Ok, thanks for the feedback. I'll try to apply your method to my code and see if that works out.