Refactor method names to remove snake_case#3030
Conversation
|
This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested. If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos. For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping |
kahgoh
left a comment
There was a problem hiding this comment.
Thanks for updating the tests and adding to the Checkstyle rule! Also thanks for fixing the link in CONTRIBUTING.md!
pull request
This pull request primarily refactors the test method names to follow camelCase naming conventions, improving code readability and consistency. Additionally, it makes a minor documentation fix and updates a method name format check in the Checkstyle configuration.
Test code cleanup and consistency:
MethodNameCheckformat incheckstyle.xmlto require camelCase method names without underscores, enforcing the new convention.CONTRIBUTING.mdfor the Gradle update instructions to use the correct path.Issue: #3028
Reviewer Resources:
Track Policies