Skip to content

build(core): fail javadoc build on warnings#949

Merged
bestbeforetoday merged 1 commit into
substrait-io:mainfrom
nielspardon:javadoc-enforce-warnings
Jun 17, 2026
Merged

build(core): fail javadoc build on warnings#949
bestbeforetoday merged 1 commit into
substrait-io:mainfrom
nielspardon:javadoc-enforce-warnings

Conversation

@nielspardon

Copy link
Copy Markdown
Member

Now that all hand-written sources in :core carry complete Javadoc (added file by file in the preceding docs(core): fix javadoc in ... PRs), this enables the -Xwerror javadoc option so that the core:javadoc task fails the build on any Javadoc lint warning.

This is the final step of the Javadoc cleanup and prevents missing-Javadoc regressions from being reintroduced. Verified locally: ./gradlew core:javadoc builds successfully with -Xwerror against the current main.

🤖 Generated with AI

Now that all hand-written sources in `:core` carry complete Javadoc, enable
the `-Xwerror` javadoc option so that the `core:javadoc` task fails the build
on any Javadoc lint warning. This prevents missing-Javadoc regressions from
being reintroduced.

Signed-off-by: Niels Pardon <par@zurich.ibm.com>
@bestbeforetoday bestbeforetoday merged commit eea2c6a into substrait-io:main Jun 17, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants