Mark for removal i platform configuration factory

[vogella]

No description provided.

[eclipse-platform-bot]

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

update/org.eclipse.update.configurator/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 8bf9b9d8e2962a20cf8d24e310e1457f96ba402f Mon Sep 17 00:00:00 2001
From: Eclipse Platform Bot <platform-bot@eclipse.org>
Date: Fri, 9 May 2025 12:50:40 +0000
Subject: [PATCH] Version bump(s) for 4.36 stream


diff --git a/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF b/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF
index 5f045c7318..f1a5deb034 100644
--- a/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF
+++ b/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %pluginName
 Bundle-SymbolicName: org.eclipse.update.configurator; singleton:=true
-Bundle-Version: 3.5.600.qualifier
+Bundle-Version: 3.5.700.qualifier
 Bundle-Activator: org.eclipse.update.internal.configurator.ConfigurationActivator
 Bundle-Vendor: %providerName
 Bundle-Localization: plugin
-- 
2.49.0

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

[github-actions]

Test Results

 1 764 files  ±0   1 764 suites  ±0   1h 27m 45s ⏱️ -53s
 4 381 tests ±0   4 357 ✅ ±0   24 💤 ±0  0 ❌ ±0 
13 143 runs  ±0  12 976 ✅ ±0  167 💤 ±0  0 ❌ ±0 

Results for commit a39ce19. ± Comparison against base commit d651768.

♻️ This comment has been updated with latest results.

[vogella]

Does anyone know what to do to solve the Javadoc warnings?

[Check warning on line 21 in 15:06:26.594 [WARNING] /home/jenkins/agent/workspace/eclipse.platform_PR-1867/update/org.eclipse.update.configurator/src/org/eclipse/update/internal/configurator/BundleGroupComponent.java](https://github.com/eclipse-platform/eclipse.platform/pull/1867/files#annotation_34551717561) 

@jenkins-eclipse-platform
jenkins-eclipse-platform
/ JavaDoc
-

NORMAL:
[removal] IPlatformConfiguration in org.eclipse.update.configurator has been deprecated and marked for removal

This points to an import statement and I see no warning in my IDE.

[vogella]

@fedejeanne do you know how I can remove the Javadoc errors? This is similar to what you might have seen in eclipse-platform/eclipse.platform.ui#2966

[fedejeanne]

@fedejeanne do you know how I can remove the Javadoc errors? This is similar to what you might have seen in eclipse-platform/eclipse.platform.ui#2966

I don't I had warnings in the imports, I only had warnings in the usage of the methods (they were not imported statically so it was a different scenario).

I also don't see a warning in my IDE and can't find any configuration to activate such warning. The only way I know to get rid of such warnings is to not have the imports i.e. use fully qualified names. I took the liberty of do that for BundleGroupComponent in b07a32f, I hope that's OK. If that works and you like the final state of the class, you could try the same in the other classes. Otherwise just revert to your last version, no harm no foul :-)

[vogella]

Thanks @fedejeanne the removal of the import is a great idea.

[vogella]

Thanks @fedejeanne for the help

[fedejeanne]

My pleasure :-)

Sorry for the late response, I finally managed to stay offline for the rest of the weekend after my last comment (it took some convincing though 😅 )

Have a nice day!