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

[wombat] migrate off VoltageController #70

Merged
merged 4 commits into from
Jan 9, 2024

Conversation

spacey-sooty
Copy link
Member

@spacey-sooty spacey-sooty commented Jan 9, 2024

What this PR does
Migrates off VoltageController to MotorController from WPILib

What does it fix/add?
Resolves #69

@spacey-sooty spacey-sooty requested a review from a team as a code owner January 9, 2024 08:24
@spacey-sooty spacey-sooty merged commit 20e6460 into master Jan 9, 2024
4 checks passed
@spacey-sooty spacey-sooty deleted the spacey-sooty/migrate-off-voltage-controller branch January 9, 2024 08:55
spacey-sooty added a commit that referenced this pull request Jan 10, 2024
spacey-sooty added a commit that referenced this pull request Jan 10, 2024
This reverts commit 20e6460.

* Revert [wombat] migrate off VoltageController (#70)

* fix swerve build

* actually fix it
spacey-sooty added a commit that referenced this pull request Jan 11, 2024
spacey-sooty added a commit that referenced this pull request Jan 11, 2024
Superbro525Alt pushed a commit to Superbro525Alt/2024-Crescendo that referenced this pull request Jan 11, 2024
spacey-sooty added a commit to prawny-boy/2024-CrescendoSeanChan that referenced this pull request Jan 16, 2024
* [wombat] migrate off VoltageController

* foramtted

* fixed -> calls

* rename to motorController
spacey-sooty added a commit to prawny-boy/2024-CrescendoSeanChan that referenced this pull request Jan 16, 2024
…nFRC#79)

This reverts commit 20e6460.

* Revert [wombat] migrate off VoltageController (CurtinFRC#70)

* fix swerve build

* actually fix it
spacey-sooty added a commit to prawny-boy/2024-CrescendoSeanChan that referenced this pull request Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate to WPILib MotorController class instead of VoltageController
2 participants