-
Notifications
You must be signed in to change notification settings - Fork 1
Board Interface V2 #216
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
Board Interface V2 #216
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request introduces Board Interface V2, which expands the IBoard interface to support multiple buttons and LEDs beyond the basic RGB LEDs and reset button. The changes rename the existing button getter for clarity and add support for three additional buttons (A, B, C) and three additional LEDs (A, B, C).
- Renames
getButton()togetButtonReset()to clarify it refers to the reset button - Adds three new button drivers (A, B, C) with corresponding getter methods
- Adds three new LED drivers (A, B, C) with corresponding getter methods
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| lib/HALInterfaces/src/IBoard.h | Updates the IBoard interface to add getButtonReset(), getButtonA(), getButtonB(), getButtonC(), getLedA(), getLedB(), and getLedC() methods |
| lib/HALTargetCommon/src/Board.h | Implements the new interface methods, adds includes for the new button/LED classes, and declares member variables for the new drivers |
| lib/HALTargetCommon/src/Board.cpp | Initializes the new button and LED member variables in the constructor |
| lib/HALSim/src/Board.h | Partially updates the simulation board implementation by renaming the m_button member variable to m_buttonReset |
| lib/APPLineFollower/src/States/StartupState.cpp | Updates caller to use the renamed getButtonReset() method |
| lib/APPLineFollower/src/States/ReleaseTrackState.cpp | Updates caller to use the renamed getButtonReset() method |
| lib/APPLineFollower/src/States/ReadyState.cpp | Updates caller to use the renamed getButtonReset() method |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.