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

Make controllers modular in robotContainer-- testing has never been easier!1! #113 #114

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

GalexY727
Copy link
Member

resolves #113
This one's nice and 🦐ple

@GalexY727 GalexY727 added enhancement New feature or request good first issue Good for newcomers teleop Something that needs to get done for the Teleop period of the match labels Feb 2, 2024
@GalexY727 GalexY727 self-assigned this Feb 2, 2024
Copy link
Contributor

@derp00monsta derp00monsta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like how you used controller for everything instead of just driver & operator.

@derp00monsta
Copy link
Contributor

@GalexY727 I think everything is good but I'm a noob so I'm going to still ask before I merge something to the grand MAIN branch. Can I merge? :D

@Jacob1010-h
Copy link
Member

Yes !! You should be good to go! As long as everything looks good by u

@derp00monsta derp00monsta merged commit 71efef5 into main Feb 2, 2024
1 check passed
@derp00monsta derp00monsta deleted the modular-controllers branch February 2, 2024 18:36
@Patribots4738 Patribots4738 deleted a comment from Jacob1010-h Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers teleop Something that needs to get done for the Teleop period of the match
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Modular Controllers
3 participants