romani commented on issue #18: [MCHECKSTYLE-381] remove call to checker.setClassLoader(). URL: https://github.com/apache/maven-checkstyle-plugin/pull/18#issuecomment-569548423 > You will start seeing warnings about some internal unused methods. it is better than live in illusions that classpath is actually helping chekstyle to do validation. Classpath functionality was completely removed in 8.28 (released today), it was very unreliable in validation from the beginning. So users unlikely happy with classpath related features. > I'd love to hear why to implement that warning. I am not ready to provide good reasons. If you not feel that it is good - I trust you. Lets proceed without warning. I just try to send a message that any attempts to provide classpath to plugin will be ignored by library/validator. I thought in some discussion that you are in favor of deprecation process, I agree with you to some extend, we need to give signal to user that classpath is deprecated (Checks do not use classpath).
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
