-
Notifications
You must be signed in to change notification settings - Fork 142
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
PMD check disabled for entire eo-runtime
#3160
Comments
@c71n93 thanks for the report, here is a feedback: Problems
SuggestionImprove the bug report title to a concise, direct statement like "PMD Check Disabled for Entire eo-runtime". Please fix the bug report in order it to get resolved faster. |
eo-runtime
@c71n93 we just don't know how to enable it. If you know, please try |
@yegor256 Did you mean this (#3171 (comment)) problem in your comment? |
Check out this line:
eo/eo-runtime/pom.xml
Line 327 in f184d9a
Do we really need to disable PMD check for the whole
eo-runtime
? We should ignore eo-runtime/src/main/java/EOorg because of upper case in package name and$
in class names, but should we ignore the rest ofeo-runtime
?There would be a lot of PMD violations for classes from eo-runtime/src/main/java/org/eolang if we turned PMD on.
@yegor256 WDYT?
The text was updated successfully, but these errors were encountered: