Skip to content

Issue #179 Fix: Add more command names#226

Open
Mehvoy wants to merge 2 commits intomainfrom
Issue-#179-Fix
Open

Issue #179 Fix: Add more command names#226
Mehvoy wants to merge 2 commits intomainfrom
Issue-#179-Fix

Conversation

@Mehvoy
Copy link
Copy Markdown
Contributor

@Mehvoy Mehvoy commented Apr 14, 2026

Named the commands listed in issue #179

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds descriptive names to various robot commands across multiple subsystems using the .withName() method, which enhances telemetry and debugging. A review comment points out an incorrect 180-degree offset in the launcherSimCommand and inconsistent indentation, providing a suggestion to align with repository rules.

Comment thread src/main/java/frc/robot/subsystems/auto/AutoLogic.java
Comment thread src/main/java/frc/robot/Controls.java Outdated
jamesdooley4
jamesdooley4 previously approved these changes Apr 14, 2026
Copy link
Copy Markdown
Member

@jamesdooley4 jamesdooley4 left a comment

Choose a reason for hiding this comment

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

Should delete the double-set of the pointFacingJoystick name.

@jamesdooley4 jamesdooley4 changed the title Issue #179 Fix Issue #179 Fix: Add more command names Apr 14, 2026
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.

2 participants