kevinjqliu opened a new pull request, #16177:
URL: https://github.com/apache/iceberg/pull/16177
This adds a `verifyRuntimeDepsApplied` task that fails CI if a subproject is
missing
`runtime-deps.gradle` and is not in an explicit exclusion list. This
prevents new
shadow JAR modules from being added without dependency tracking.
### Changes
- Added `verifyRuntimeDepsApplied` task to `build.gradle` with an exclusion
list of
modules that don't ship fat JARs
- Wired it as a dependency of `checkAllRuntimeDeps` so it runs in CI
automatically
- Updated workflow comment in `runtime-deps.gradle`
### How it works
Every subproject must either:
1. Apply `runtime-deps.gradle` (produces a `checkRuntimeDeps` task), or
2. Be listed in the exclusion set
If a new module is added without doing either, CI fails with a message naming
the uncovered module and instructions to fix it.
### Testing
- `./gradlew checkAllRuntimeDeps -q -DallModules=true` passes
- Verified the task catches missing modules by temporarily removing
`runtime-deps.gradle` from `aws-bundle`
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
