Skip to content

Moved con operator declaration closer to the con driver declaration - #68

Merged
TaylerUva merged 2 commits into
mainfrom
53-move-configureoperatorbindings-next-to-configuredrivebindings
Jan 19, 2026
Merged

TaylerUva merged 2 commits into
mainfrom
53-move-configureoperatorbindings-next-to-configuredrivebindings

Conversation

@S0L0GUY

@S0L0GUY S0L0GUY commented Jan 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@S0L0GUY
S0L0GUY requested a review from a team as a code owner January 19, 2026 19:34
Copilot AI review requested due to automatic review settings January 19, 2026 19:34
@S0L0GUY S0L0GUY linked an issue Jan 19, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR reorganizes the RobotContainer class by moving the configOperatorBindings() method to appear immediately after configDriverBindings(), grouping related configuration methods together. The move improves code organization by placing configuration methods in the same order as they are called in the constructor.

Changes:

  • Relocated configOperatorBindings() method from after getAutonomousCommand() to after configDriverBindings()

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TaylerUva
TaylerUva merged commit ef1b882 into main Jan 19, 2026
1 check passed
@TaylerUva
TaylerUva deleted the 53-move-configureoperatorbindings-next-to-configuredrivebindings branch January 19, 2026 19:46
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.

Move configureOperatorBindings next to configureDriveBindings

3 participants