gnodet opened a new pull request, #524: URL: https://github.com/apache/maven-resources-plugin/pull/524
Follow-up cleanup on top of #523 (bump to 4.0.0-rc-7). ## Root cause `TestResourcesMojo` and its parent `ResourcesMojo` both declared a `private boolean skip` field. Due to a bug in `EnhancedCompositeBeanHelper.buildFieldCache()` (apache/maven#12626), the parent field won overwrote the child's in the DI cache, so configuring `<skip>true</skip>` on the `testResources` goal had no effect — the child's field was never written to. The workaround in `execute()` was to read **both** fields: `if (skip || isSkip())`. ## Fix apache/maven#12626 (`putIfAbsent` instead of `put` in `buildFieldCache()`) was merged 2026-08-28 and ships in 4.0.0-rc-7, which is exactly the version bumped in #523. With rc-7, `TestResourcesMojo.skip` is now correctly injected by the core configurator. The `|| isSkip()` fallback and its 10-line explanatory comment can go away. Fixes #442 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
