Skip to content

Conversation

@rift10
Copy link
Contributor

@rift10 rift10 commented Nov 6, 2025

Author - rebecca

Description

  • changed folder organization of commands
  • aligned existing commands to commands list

Checklist:

Mark with X's once you have tested

  • Follows the style guidelines of this project
  • Builds with no errors off robot
  • Main branch merged into this branch
  • Changes generate no new errors or warnings
  • Code deploys and runs as expected on robot
  • Commented code, especially in hard-to-understand areas

@rift10 rift10 requested a review from worker-bob November 6, 2025 06:11
Copy link
Contributor

@worker-bob worker-bob left a comment

Choose a reason for hiding this comment

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

Also, can you just rename all the goals? we have some named intake, to_intake. Just name it to intake, and if its like TO_SCOREL1, just name it L1

Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't we move this file to the compound folder? Also could probably be renamed to something else

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i was thinking the compound folder should be the top level sequential commands but we could change that if you want

Copy link
Contributor Author

Choose a reason for hiding this comment

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

also what would you rename it to then

@rift10 rift10 merged commit b64bc62 into main Nov 7, 2025
2 checks passed
@rift10 rift10 deleted the fix-commands branch November 7, 2025 01:33
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