slawekjaranowski commented on code in PR #198: URL: https://github.com/apache/maven-enforcer/pull/198#discussion_r1051445542
########## enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractBanDependencies.java: ########## @@ -18,22 +18,15 @@ */ Review Comment: I must agree with Andrzej if someone use internal implementation, we should not stop our improvements in such case. Documentation say that api should be used nothing else. Yes we should think to hide internal classes when we can, but sometime it is not possible for technical reason. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
