-
Notifications
You must be signed in to change notification settings - Fork 127
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
#1249: It should align inner classes in tests #3243
Conversation
@maxonfjvipon please review |
eo-maven-plugin/src/main/resources/org/eolang/maven/pre/rename-tests-inners.xsl
Show resolved
Hide resolved
eo-maven-plugin/src/main/resources/org/eolang/maven/pre/remove-high-level-inner-classes.xsl
Outdated
Show resolved
Hide resolved
eo-maven-plugin/src/main/resources/org/eolang/maven/pre/remove-high-level-inner-classes.xsl
Outdated
Show resolved
Hide resolved
eo-maven-plugin/src/main/resources/org/eolang/maven/pre/align-test-classes.xsl
Outdated
Show resolved
Hide resolved
eo-maven-plugin/src/main/resources/org/eolang/maven/pre/align-test-classes.xsl
Outdated
Show resolved
Hide resolved
@yegor256 please check |
4 similar comments
@yegor256 please check |
@yegor256 please check |
@yegor256 please check |
@yegor256 please check |
@yegor256 please take a look |
@rultor merge |
@levBagryansky @yegor256 Oops, I failed. You can see the full log here (spent 44min)
|
@levBagryansky Thanks for the contribution! You've earned +5 points for this (+20 as a basis, -7 for too many hits-of-code (240), -19 for 19 comments, +11 it is too few). Please, keep them coming. Your running balance is +47. |
Closes #1249
I added 2 transformations:
I also changed the naming convention of nested classes
PR-Codex overview
This PR optimizes the Maven plugin by aligning test classes, removing high-level inner classes, and updating XSL files.
Detailed summary