Skip to content

Skip javadoc generation for test bundles - #2913

Closed
vogella wants to merge 2 commits into
eclipse-platform:masterfrom
vogella:skip-javadoc-for-test-bundles
Closed

Skip javadoc generation for test bundles#2913
vogella wants to merge 2 commits into
eclipse-platform:masterfrom
vogella:skip-javadoc-for-test-bundles

Conversation

@vogella

@vogella vogella commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The javadoc profile that the Jenkins job enables generates and jars API docs for every module in the reactor, including the 23 test bundles. Nobody publishes those docs and the Jenkinsfile archives only logs, so the jars are built and then thrown away.

Javadoc stays enabled for the published bundles. Its doclint HTML and syntax checks are the only validation of doc comment markup in CI, since the GitHub Actions builds do not pass -Pjavadoc at all.

Same change as eclipse-platform/eclipse.platform.ui#4358. Most test bundles here are pomless and get the property via build.properties; the five with an explicit pom.xml set maven.javadoc.skip directly, since the pom.model.property. mechanism only applies to pomless projects.

vogella and others added 2 commits September 8, 2026 08:52
The javadoc profile the Jenkins job runs generates and jars API docs for every
module, including the 23 test bundles whose docs are never published and whose
jars the build throws away.

Javadoc stays enabled for the published bundles, where doclint is the only
check on HTML and syntax in doc comments that CI has.

Same change as eclipse-platform/eclipse.platform.ui#4358. Pomless bundles get
the property through build.properties, the five test bundles with an explicit
pom.xml set it directly.

Assisted-by: multiple AI agents and layers of automated tooling 🤖
@eclipse-platform-bot

Copy link
Copy Markdown
Contributor

This pull request changes some projects for the first time in this development cycle.
Therefore the following files need a version increment:

runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF

An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch.

Git patch
From 77f1c25a555ec5110df787f1318c6cbe7a9aa455 Mon Sep 17 00:00:00 2001
From: Eclipse Platform Bot <platform-bot@eclipse.org>
Date: Tue, 8 Sep 2026 07:00:34 +0000
Subject: [PATCH] Version bump(s) for 4.42 stream


diff --git a/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF b/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF
index 63dba3b3e1..624a2e0eb5 100644
--- a/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF
+++ b/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: Eclipse Core Tests Harness
 Bundle-SymbolicName: org.eclipse.core.tests.harness;singleton:=true
-Bundle-Version: 3.17.500.qualifier
+Bundle-Version: 3.17.600.qualifier
 Bundle-Vendor: Eclipse.org
 Export-Package: org.eclipse.core.tests.harness;version="2.0",
  org.eclipse.core.tests.harness.session
-- 
2.55.0

Further information are available in Common Build Issues - Missing version increments.

@vogella
vogella requested a review from akurtakov September 8, 2026 07:27
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Test Results

    54 files  ±0      54 suites  ±0   56m 11s ⏱️ - 1m 55s
 4 767 tests ±0   4 745 ✅ ±0   22 💤 ±0  0 ❌ ±0 
12 192 runs  ±0  12 039 ✅ ±0  153 💤 ±0  0 ❌ ±0 

Results for commit a3c9a4c. ± Comparison against base commit a68c37a.

@vogella vogella closed this Sep 8, 2026
@vogella
vogella deleted the skip-javadoc-for-test-bundles branch September 8, 2026 18:44
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.

3 participants