Skip to content

Electric#383

Draft
stephenjust wants to merge 4 commits intomainfrom
electric
Draft

Electric#383
stephenjust wants to merge 4 commits intomainfrom
electric

Conversation

@stephenjust
Copy link
Copy Markdown
Contributor

@stephenjust stephenjust commented Jan 21, 2026

Why are we doing this? Merges the electrical contract into the file Contract2026.java. Also, provides contract testing and automatic report generation.

Asana task URL: None

Whats changing? Contgract2026.java is changing along with a new report generation. Also gradlew.

Questions/notes for reviewers

Notes:

  1. run with at project root: .\gradlew electricalReport --console=plain > electrical-report.txt
  2. Don't need to pipe output into the report
  3. The cycle between electrical team and prog team should be simpler where:
  • electrical team modifies Contract2026.java and checks back into github
  • electrical team generates is electrical report
  • programming team gets the best electrical contract values without any hand translation

How this was tested

  • tested on robot
  • tested in simulator
  • unit tests added
  • [ X ] Tested in branch running against the existing Contract2026.java code

Video/screenshots (from simulator or live robot)


PR feedback legend

Symbol Meaning
⭐ ⭐ ⭐ must be addressed
⭐ ⭐ should be addressed
something to consider, a good idea

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