Skip to content

State machine update - #32

Merged
TaylerUva merged 3 commits into
mainfrom
state-machine-update
May 30, 2026
Merged

State machine update#32
TaylerUva merged 3 commits into
mainfrom
state-machine-update

Conversation

@Wu-Fan-529

Copy link
Copy Markdown
Member

No description provided.

@Wu-Fan-529
Wu-Fan-529 requested a review from a team as a code owner May 30, 2026 19:49
Copilot AI review requested due to automatic review settings May 30, 2026 19:49

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

Renames several robot state machine entries and their corresponding command classes to align names across the diagram and Java code. The PREP_DEPOT and PREP_NON_OUTPOST enum values are removed, and the PrepDepot/PrepNonOutpost/PrepOutpost command classes are renamed to PrepHub/PrepCorner/PrepTower. The robot.tldr state diagram is updated (mostly) to reflect the new names, plus some unrelated tldraw editor metadata churn.

Changes:

  • Remove PREP_DEPOT and PREP_NON_OUTPOST from RobotState enum.
  • Rename command classes PrepDepotPrepHub, PrepNonOutpostPrepCorner, PrepOutpostPrepTower (file/class only — Javadoc and method bodies otherwise unchanged).
  • Update robot.tldr shape labels to the new names; reformat empty lifecycle method bodies in the renamed commands.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/main/java/frc/robot/subsystems/StateMachine.java Drops PREP_DEPOT and PREP_NON_OUTPOST from the RobotState enum.
src/main/java/frc/robot/commands/states/PrepHub.java Renames PrepDepot class/constructor to PrepHub; Javadoc still says "PrepDepot".
src/main/java/frc/robot/commands/states/PrepCorner.java Renames PrepNonOutpost class/constructor to PrepCorner; Javadoc still says "PrepNonOutpost".
src/main/java/frc/robot/commands/states/PrepTower.java Renames PrepOutpost class/constructor to PrepTower; Javadoc still says "PrepOutpost".
robot.tldr Updates several state diagram labels to new names; re-adds an PrepOpponentToAlliance shape but with a misspelling, plus unrelated tldraw editor state changes.
Comments suppressed due to low confidence (3)

src/main/java/frc/robot/commands/states/PrepHub.java:10

  • The Javadoc comment still references the old class name PrepDepot. Since the class has been renamed to PrepHub, this comment should be updated to match.
    src/main/java/frc/robot/commands/states/PrepCorner.java:10
  • The Javadoc comment still references the old class name PrepNonOutpost. Since the class has been renamed to PrepCorner, this comment should be updated to match.
    src/main/java/frc/robot/commands/states/PrepTower.java:10
  • The Javadoc comment still references the old class name PrepOutpost. Since the class has been renamed to PrepTower, this comment should be updated to match.

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

Comment thread robot.tldr
@TaylerUva
TaylerUva enabled auto-merge (squash) May 30, 2026 20:17
@TaylerUva
TaylerUva merged commit 2b473c0 into main May 30, 2026
2 checks passed
@TaylerUva
TaylerUva deleted the state-machine-update branch May 30, 2026 20:17
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.

3 participants