Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move to inlined commands and command factories #28

Open
TylerSeiford opened this issue Oct 21, 2023 · 0 comments
Open

Move to inlined commands and command factories #28

TylerSeiford opened this issue Oct 21, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@TylerSeiford
Copy link
Contributor

As described in the WPILib docs, the preferred/best practice method to structure a robot program is to use inline commands rather than sub-classing the Command class.

In addition to moving commands from their own class to an inline command, it is also recommended to do so within the subsystem as an instance command factory method.

This is a paradigm change from the current method we use to code robots. Instead of writing a subsystem and exposing functions which can be called from commands, we write a subsystem and expose commands which can be directly bound to triggers.

@TylerSeiford TylerSeiford added enhancement New feature or request good first issue Good for newcomers and removed good first issue Good for newcomers labels Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Open
Development

No branches or pull requests

1 participant