Skip to content

(#233) changed auto naming to have alliance and mirror first - #235

Merged
Tyrone337 merged 2 commits into
mainfrom
233-auto-selection
Oct 14, 2026
Merged

Tyrone337 merged 2 commits into
mainfrom
233-auto-selection

Conversation

@Tyrone337

Copy link
Copy Markdown
Contributor
  • also commented out conservative autos

- also commented out conservative autos
@Tyrone337
Tyrone337 requested a review from thetaback October 12, 2026 23:50

@thetaback thetaback 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.

Looks mostly good, just one small fix. also can you confirm this works in sim as intended?

Comment thread src/main/java/frc/robot/auto/Autos.java Outdated
@Tyrone337

Copy link
Copy Markdown
Contributor Author
image here's a screenshot of the updated auto selections in elastic.

@Tyrone337
Tyrone337 requested review from godmar and thetaback October 14, 2026 22:39

@godmar godmar 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.

LGTM

@godmar
godmar dismissed thetaback’s stale review October 14, 2026 22:54

I reviewed this

@Tyrone337
Tyrone337 merged commit e167ac3 into main Oct 14, 2026
3 checks passed
@Tyrone337
Tyrone337 deleted the 233-auto-selection branch October 14, 2026 22:55
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.

4 participants