gnodet commented on PR #243: URL: https://github.com/apache/maven-clean-plugin/pull/243#issuecomment-2774647889
> > The fact that the Maven 4 API exposes `PathMatcher` while the plexus scanner uses String[] is a problematic. > > We can also revert this API to plain strings, and complete them by a `getMatcher()` method which return a single, global `Optional<PathMatcher>`. The `Selector` class implements `PathMatcher`, so it can be seen as a single matcher doing all the includes / excludes work. I think that would make sense. The point is that the resource plugin can also be configured from the plugin, so with `Path` or `String`, but definitely not `PathMatcher`. -- 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
