Run CI Javadocs with Java 25

[shai-almog]

Motivation

• Ensure the Javadoc generation step actually runs using Java 25 in CI rather than only being compatible, so doclint behavior matches modern JDKs.
• Suppress verbose doclint warnings that clutter CI logs on newer JDKs like Java 25.
• Keep Javadoc output consistent across JDK versions without changing source code.

Description

• Added a Set up Java 25 for JavaDocs step to .github/workflows/pr.yml using actions/setup-java@v4 with distribution: temurin and java-version: '25' conditioned on matrix.java-version == 8.
• Updated the Build JavaDocs step to invoke javadoc with -Xdoclint:none -quiet and preserved the || true behavior to avoid failing the job on warnings.
• The workflow still prepares the temp source dirs and packages the generated docs into javadocs.zip as before.

Testing

• No automated tests were executed locally because this is a CI workflow change and behavior will be validated by CI.
• Existing CI unit test and packaging steps remain unchanged and will continue to run in CI.

---

Codex Task

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2282 total, 0 failed, 0 skipped
• 📊 Line coverage: 52.87% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.io.rest.RequestBuilder$FetchAsPropertyListActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$FetchAsPropertiesActionListener – 0.00%
    • com.codename1.ui.Display$EdtException – 0.00%
    • com.codename1.ui.BrowserWindow$BrowserForm – 0.00%
    • com.codename1.ui.plaf.CSSBorder$LinearGradient – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsStringAsyncImplActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsJsonMapActionListener – 0.00%
    • com.codename1.payment.Purchase$SynchronizeReceiptsSyncRunnable – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsBytesAsyncImplActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$FetchAsJsonMapActionListener – 0.00%

Static Analysis

• ❌ SpotBugs: 106 findings (Normal: 71, Low: 35) [HTML preview] [Download]
  • Top findings
    • Normal: CodenameOne/src/com/codename1/capture/Capture.java:262 – Naked notify in com.codename1.capture.Capture$CallBack.actionPerformed(ActionEvent) (rule: NN_NAKED_NOTIFY)
    • Normal: CodenameOne/src/com/codename1/charts/ChartComponent.java:682 – Return value of com.codename1.charts.renderers.XYMultipleSeriesRenderer.isPanEnabled() ignored, but method has no side effect (rule: RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT)
    • Normal: CodenameOne/src/com/codename1/charts/ChartComponent.java:770 – Return value of com.codename1.charts.renderers.XYMultipleSeriesRenderer.isZoomEnabled() ignored, but method has no side effect (rule: RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT)
    • Normal: CodenameOne/src/com/codename1/charts/compat/Paint.java:38 – Unused field: com.codename1.charts.compat.Paint.g (rule: UUF_UNUSED_FIELD)
    • Normal: CodenameOne/src/com/codename1/charts/views/BarChart.java:383 – Unused field: com.codename1.charts.views.BarChart$Point.seriesIndex (rule: UUF_UNUSED_FIELD)
    • …and 101 more
• ❌ PMD: 2555 findings (P2: 1, P3: 486, P4: 2068) [HTML preview] [Download]
  • Top findings
    • P2: CodenameOne/src/com/codename1/ui/Display.java:1040 – Avoid using a branching statement as the last in a loop. (rule: AvoidBranchingStatementAsLastInLoop)
    • P3: CodenameOne/src/com/codename1/charts/views/LineChart.java:208 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • P3: CodenameOne/src/com/codename1/components/RSSReader.java:500 – Overriding method merely calls super (rule: UselessOverridingMethod)
    • P3: CodenameOne/src/com/codename1/facebook/Album.java:198 – Overriding method merely calls super (rule: UselessOverridingMethod)
    • P3: CodenameOne/src/com/codename1/facebook/FaceBookAccess.java:1312 – Unnecessary semicolon (rule: UnnecessarySemicolon)
    • …and 2550 more
• ❌ Checkstyle: 220847 findings (Warning: 220847) [HTML preview] [Download]
  • Top findings
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:30:1 – Extra separation in import group before 'java.io.IOException' (rule: CustomImportOrderCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:32 – First sentence of Javadoc is missing an ending period. (rule: SummaryJavadocCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:36 – Line is longer than 100 characters (found 143). (rule: LineLengthCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:39:5 – 'member def modifier' has incorrect indentation level 4, expected level should be 2. (rule: IndentationCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:40:5 – 'member def modifier' has incorrect indentation level 4, expected level should be 2. (rule: IndentationCheck)
    • …and 220842 more

Generated automatically by the PR CI workflow.