Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#533] code review #535

Merged
merged 7 commits into from
Apr 12, 2024
Merged

[#533] code review #535

merged 7 commits into from
Apr 12, 2024

Conversation

MattD8957
Copy link
Contributor

Closes #533

@MattD8957 MattD8957 added cleanup Something is messy and needs to be fixed auton Code related to Auton labels Apr 12, 2024
@MattD8957 MattD8957 added this to the [COMP] Worlds milestone Apr 12, 2024
@MattD8957 MattD8957 requested a review from a team April 12, 2024 01:22
@MattD8957 MattD8957 linked an issue Apr 12, 2024 that may be closed by this pull request
Copy link
Member

@WindowsVistaisCool WindowsVistaisCool left a comment

Choose a reason for hiding this comment

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

if thunder changes are okay, fix your linter, then im happy

@@ -25,5 +25,6 @@
}
},
],
"java.test.defaultConfig": "WPIlibUnitTests"
"java.test.defaultConfig": "WPIlibUnitTests",
"java.format.settings.url": ".vscode/eclipse-java-google-style.xml"
Copy link
Member

Choose a reason for hiding this comment

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

this should be set to .github/sun_checks.xml

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@WindowsVistaisCool You need to change the online version then, because this is the fix for redhat formatter

src/main/java/frc/robot/Constants.java Show resolved Hide resolved
@MattD8957
Copy link
Contributor Author

I'm still cleaning, also the LED thing got fixed. It needs to be that or you need to change the one online because those settings are the ones that make the formater funny.

@MattD8957 MattD8957 merged commit acd4306 into main Apr 12, 2024
5 checks passed
@MattD8957 MattD8957 deleted the 533-code-review branch April 12, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auton Code related to Auton cleanup Something is messy and needs to be fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Code review
3 participants