Skip to content

Conversation

@Maillman
Copy link
Contributor

Overview

Updates the checkstyle jar with an improvement to the commented code check.

For more information regarding this improvement, read softwareconstruction240/checkstyle#11

@Maillman Maillman requested a review from a team September 10, 2025 20:23
Copy link
Contributor

@ZakkeryDaRebel ZakkeryDaRebel left a comment

Choose a reason for hiding this comment

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

If I'm not mistaken, this checkstyle will check student's commented out code, and dock code quality if it is too much commented out code, and if the code doesn't follow regex naming conventions? Or is there something else it is doing?

@19mdavenport
Copy link
Contributor

@Maillman you may need to change a string that has the name of the file in it somewhere

@ZakkeryDaRebel It's the entire code quality checker (including duplicate code checks, etc.). This version is just better at determining what comments are code

@Maillman Maillman merged commit 8aa35b1 into main Sep 15, 2025
3 checks passed
@Maillman Maillman deleted the update-checkstyle branch September 15, 2025 19:24
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