sertain / sertain-legacy

:robot: Write more for your robot with less
https://sertain.github.io/javadocs
MIT License
9 stars 3 forks source link

Make names of commands more intuitive #51

Closed gnawinganimal closed 5 years ago

gnawinganimal commented 5 years ago

Names like CommandBridgeMirror and CommandBridge should not be exposed in the API, because they describe how they work and not what they should be used for. I think that renaming them to CommandLifecycle and CommandBase makes more sense to an end user of sertain. Whaddya say?

andrewda commented 5 years ago

Good points on the naming. Let's test at robotics to ensure none of these API changes affect anything.

gnawinganimal commented 5 years ago

I'll be there on Friday 👍