elharo commented on PR #447: URL: https://github.com/apache/maven-dependency-plugin/pull/447#issuecomment-2429159956
Yes, I'm not sure why this was done like this in the first place. This has been around since 2.0 so who knows? I am confident the way it was done is buggy, so this tries to repair as much of it as I easily could. I noticed because the tests mixed up two different silent log classes. Someone wasn't following test first practices when they wrote this. If we're comfortable with some incompatibility, then we could mke the silent parameter and associated methods noops. -- 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