Skip to content

add a mixin to make test method names be displayed in snake_case#4463

Merged
TarLaboratories merged 1 commit intoGregTechCEu:1.20.1from
screret:sc/make-test-functions-always-snake-case
Jan 13, 2026
Merged

add a mixin to make test method names be displayed in snake_case#4463
TarLaboratories merged 1 commit intoGregTechCEu:1.20.1from
screret:sc/make-test-functions-always-snake-case

Conversation

@screret
Copy link
Copy Markdown
Contributor

@screret screret commented Jan 12, 2026

...even if the actual method name is in camelCase

What

The test methods being displayed as blocked_by_ldlib_weirdness_probably_testadvancedactivitydetectorcover and the like bothered me so I wrote this simple mixin to fix that :3

@screret screret requested a review from a team as a code owner January 12, 2026 22:46
@TarLaboratories TarLaboratories added Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. type: tests Test related changes labels Jan 13, 2026
Copy link
Copy Markdown
Contributor

@TarLaboratories TarLaboratories left a comment

Choose a reason for hiding this comment

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

LGTM

@TarLaboratories TarLaboratories added the ignore changelog PR should not be added to the changelog. label Jan 13, 2026
@TarLaboratories TarLaboratories merged commit ba3bec0 into GregTechCEu:1.20.1 Jan 13, 2026
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 ignore changelog PR should not be added to the changelog. Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. type: tests Test related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants