Skip to content

Do not ship javadoc web fonts for individual modules#11099

Merged
artembilan merged 1 commit into
spring-projects:mainfrom
bclozel:javadoc-fonts
Jun 11, 2026
Merged

Do not ship javadoc web fonts for individual modules#11099
artembilan merged 1 commit into
spring-projects:mainfrom
bclozel:javadoc-fonts

Conversation

@bclozel

@bclozel bclozel commented Jun 11, 2026

Copy link
Copy Markdown
Member

Web fonts use 4MB of space per "-javadoc.jar" file, for each module shipped by the project. While those are useful when browsing the javadoc, this project aggregates all javadocs under a single JAR and hosts it on doc.spring.io.

This commit skips the inclusion for the individual modules' javadoc since those are mainly used by IDEs.
The web fonts are still available for the aggregated version.

This is worth backporting to any branch building on Java 23+.

Web fonts use 4MB of space per "-javadoc.jar" file, for each module
shipped by the project. While those are useful when browsing the
javadoc, this project aggregates all javadocs under a single JAR and
hosts it on doc.spring.io.

This commit skips the inclusion for the individual modules' javadoc
since those are mainly used by IDEs.
The web fonts are still available for the aggregated version.

Signed-off-by: Brian Clozel <brian.clozel@broadcom.com>
@artembilan artembilan added this to the 7.1.1 milestone Jun 11, 2026
@artembilan artembilan merged commit ba7cd23 into spring-projects:main Jun 11, 2026
3 checks passed
spring-builds pushed a commit that referenced this pull request Jun 11, 2026
Fixes: #11099

Web fonts use 4MB of space per "-javadoc.jar" file, for each module
shipped by the project. While those are useful when browsing the
javadoc, this project aggregates all javadocs under a single JAR and
hosts it on doc.spring.io.

This commit skips the inclusion for the individual modules' javadoc
since those are mainly used by IDEs.
The web fonts are still available for the aggregated version.

Signed-off-by: Brian Clozel <brian.clozel@broadcom.com>

(cherry picked from commit ba7cd23)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants