Skip to content

Enable failOnWarnings in maven-javadoc-plugin#1135

Merged
wendigo merged 1 commit intoairlift:masterfrom
nineinchnick:user/jwas/fail-on-warnings
Mar 10, 2026
Merged

Enable failOnWarnings in maven-javadoc-plugin#1135
wendigo merged 1 commit intoairlift:masterfrom
nineinchnick:user/jwas/fail-on-warnings

Conversation

@nineinchnick
Copy link
Contributor

@nineinchnick nineinchnick commented Mar 10, 2026

Summary

  • Add <failOnWarnings>true</failOnWarnings> to the maven-javadoc-plugin configuration

This ensures javadoc warnings are treated as errors, catching issues like
missing @param/@return tags or invalid references early in the build.

@nineinchnick nineinchnick force-pushed the user/jwas/fail-on-warnings branch from 77911ae to 85d667b Compare March 10, 2026 11:42
@nineinchnick nineinchnick changed the title Enable failOnWarning in maven-compiler-plugin Enable failOnWarnings in maven-javadoc-plugin Mar 10, 2026
@nineinchnick nineinchnick reopened this Mar 10, 2026
@nineinchnick
Copy link
Contributor Author

I haven't added any new properties to control this since air.javadoc.lint can be set to none.

<encoding>${project.build.sourceEncoding}</encoding>
<maxmemory>${air.build.jvmsize}</maxmemory>
<doclint>${air.javadoc.lint}</doclint>
<failOnWarnings>true</failOnWarnings>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we have a property to disable it back if needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wendigo wendigo merged commit 8d4279c into airlift:master Mar 10, 2026
4 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