Merge pull request #635 from bjhargrave/1.0-release

Prepare for 1.0 release

Author: bjhargrave

.github/workflows/cibuild.yml

@@ -49,8 +49,7 @@ jobs:
         os:
         - 'ubuntu-latest'
         java:
-        - '8'
-        - '11'
+        - '21'
         include:
         - os: 'ubuntu-latest'
           java: '17'

.mvn/wrapper/maven-wrapper.jar

@@ -6,13 +6,15 @@
 # "License"); you may not use this file except in compliance
 # with the License.  You may obtain a copy of the License at
 #
-#   https://www.apache.org/licenses/LICENSE-2.0
+#   http://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing,
 # software distributed under the License is distributed on an
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
 # under the License.
-distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.8.7/apache-maven-3.8.7-bin.zip
-wrapperUrl=https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.1.1/maven-wrapper-3.1.1.jar
+wrapperVersion=3.3.2
+distributionType=bin
+distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.9/apache-maven-3.9.9-bin.zip
+wrapperUrl=https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.3.2/maven-wrapper-3.3.2.jar

.mvn/wrapper/maven-wrapper.properties

@@ -30,9 +30,9 @@ org.eclipse.jdt.core.compiler.annotation.nullable.secondary=
 org.eclipse.jdt.core.compiler.annotation.nullanalysis=disabled
 org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled
 org.eclipse.jdt.core.compiler.codegen.methodParameters=do not generate
-org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.8
+org.eclipse.jdt.core.compiler.codegen.targetPlatform=17
 org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve
-org.eclipse.jdt.core.compiler.compliance=1.8
+org.eclipse.jdt.core.compiler.compliance=17
 org.eclipse.jdt.core.compiler.debug.lineNumber=generate
 org.eclipse.jdt.core.compiler.debug.localVariable=generate
 org.eclipse.jdt.core.compiler.debug.sourceFile=generate
@@ -72,7 +72,7 @@ org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=public
 org.eclipse.jdt.core.compiler.problem.localVariableHiding=ignore
 org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=warning
 org.eclipse.jdt.core.compiler.problem.missingDefaultCase=ignore
-org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=ignore
+org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=warning
 org.eclipse.jdt.core.compiler.problem.missingEnumCaseDespiteDefault=disabled
 org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=warning
 org.eclipse.jdt.core.compiler.problem.missingJavadocComments=ignore
@@ -150,7 +150,7 @@ org.eclipse.jdt.core.compiler.problem.unusedWarningToken=ignore
 org.eclipse.jdt.core.compiler.problem.varargsArgumentNeedCast=warning
 org.eclipse.jdt.core.compiler.processAnnotations=disabled
 org.eclipse.jdt.core.compiler.release=enabled
-org.eclipse.jdt.core.compiler.source=1.8
+org.eclipse.jdt.core.compiler.source=17
 org.eclipse.jdt.core.compiler.taskCaseSensitive=enabled
 org.eclipse.jdt.core.compiler.taskPriorities=NORMAL,NORMAL
 org.eclipse.jdt.core.compiler.taskTags=TODO,FIXME

bnd-plugins/org.eclipse.transformer.bnd.analyzer/.settings/org.eclipse.jdt.core.prefs

@@ -21,9 +21,9 @@
 @ExternalPlugin(name = "Transformer", objectClass = AnalyzerPlugin.class)
 public class TransformerAnalyzerPlugin extends BaseTransformerPlugin implements AnalyzerPlugin {
 	public TransformerAnalyzerPlugin() {
-		/**
-		 * We want to run before other AnalyzerPlugins so they will operate on
-		 * the transformed classes and resources.
+		/*
+		  We want to run before other AnalyzerPlugins so they will operate on
+		  the transformed classes and resources.
 		 */
 		super(-10_000);
 	}

bnd-plugins/org.eclipse.transformer.bnd.analyzer/src/main/java/org/eclipse/transformer/bnd/analyzer/TransformerAnalyzerPlugin.java

@@ -21,9 +21,9 @@
 @ExternalPlugin(name = "Transformer", objectClass = VerifierPlugin.class)
 public class TransformerVerifierPlugin extends BaseTransformerPlugin implements VerifierPlugin {
 	public TransformerVerifierPlugin() {
-		/**
-		 * We want to run after other VerifierPlugins so we will operate on the
-		 * final classes and resources.
+		/*
+		  We want to run after other VerifierPlugins so we will operate on the
+		  final classes and resources.
 		 */
 		super(10_000);
 	}

bnd-plugins/org.eclipse.transformer.bnd.analyzer/src/main/java/org/eclipse/transformer/bnd/analyzer/TransformerVerifierPlugin.java

@@ -30,9 +30,9 @@ org.eclipse.jdt.core.compiler.annotation.nullable.secondary=
 org.eclipse.jdt.core.compiler.annotation.nullanalysis=disabled
 org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled
 org.eclipse.jdt.core.compiler.codegen.methodParameters=do not generate
-org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.8
+org.eclipse.jdt.core.compiler.codegen.targetPlatform=17
 org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve
-org.eclipse.jdt.core.compiler.compliance=1.8
+org.eclipse.jdt.core.compiler.compliance=17
 org.eclipse.jdt.core.compiler.debug.lineNumber=generate
 org.eclipse.jdt.core.compiler.debug.localVariable=generate
 org.eclipse.jdt.core.compiler.debug.sourceFile=generate
@@ -72,7 +72,7 @@ org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=public
 org.eclipse.jdt.core.compiler.problem.localVariableHiding=ignore
 org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=warning
 org.eclipse.jdt.core.compiler.problem.missingDefaultCase=ignore
-org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=ignore
+org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=warning
 org.eclipse.jdt.core.compiler.problem.missingEnumCaseDespiteDefault=disabled
 org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=warning
 org.eclipse.jdt.core.compiler.problem.missingJavadocComments=ignore
@@ -150,7 +150,7 @@ org.eclipse.jdt.core.compiler.problem.unusedWarningToken=ignore
 org.eclipse.jdt.core.compiler.problem.varargsArgumentNeedCast=warning
 org.eclipse.jdt.core.compiler.processAnnotations=disabled
 org.eclipse.jdt.core.compiler.release=enabled
-org.eclipse.jdt.core.compiler.source=1.8
+org.eclipse.jdt.core.compiler.source=17
 org.eclipse.jdt.core.compiler.taskCaseSensitive=enabled
 org.eclipse.jdt.core.compiler.taskPriorities=NORMAL,NORMAL
 org.eclipse.jdt.core.compiler.taskTags=TODO,FIXME

maven-plugins/transformer-maven-plugin/.settings/org.eclipse.jdt.core.prefs

@@ -8,8 +8,6 @@ This plugin contains the following goals:
 - [`transform`](#transform-goal)
 - [`help`](#help-goal)
 
-__Note__: if you are using Java 21 or later see [Using Java 21 or later](#java-21-or-later).
-
 ## `jar` Goal
 
 The `jar` goal operates at the JAR file level and transforms the specified input artifact into a project artifact.
@@ -253,17 +251,3 @@ It is therefore necessary to configure the `maven-jar-plugin` as follows:
 
 The `help` goal displays help about the goals of the plugin.
 Call `mvn transformer:help -Ddetail=true -Dgoal=<goal-name>` to display configuration details for the specified goal.
-
-## Java 21 or later
-
-If you are using Java 21 or later you will need to configure the Maven plugin to use a specific version of the `biz.aQute.bndlib` dependency.
-
-For example:
-
-```xml
-  <dependency>
-    <groupId>biz.aQute.bnd</groupId>
-    <artifactId>biz.aQute.bndlib</artifactId>
-    <version>7.0.0</version>
-  </dependency>
-```

maven-plugins/transformer-maven-plugin/README.md

@@ -79,7 +79,7 @@ public boolean skip() {
 		// Exit without generating anything if this project is not a known
 		// packaging type. Probably it's just a parent project.
 		if (!getPackagingTypes().contains(getProject().getPackaging())) {
-			getLogger().debug("skip project with packaging=" + getProject().getPackaging());
+			getLogger().debug("skip project with packaging={}", getProject().getPackaging());
 			return true;
 		}
 

maven-plugins/transformer-maven-plugin/src/main/java/org/eclipse/transformer/maven/AbstractTransformerMojo.java

@@ -290,16 +290,16 @@ private TransformerArtifact prepareArtifactDescription(TransformerArtifact artif
 			List<Dependency> firstPass = dependencies.stream()
 				.filter(dependency -> Objects.equals(artifactDescription.getGroupId(), dependency.getGroupId())
 					&& Objects.equals(artifactDescription.getArtifactId(), dependency.getArtifactId()))
-				.collect(toList());
+				.toList();
 
 			Optional<Dependency> matchingDependency = firstPass.stream()
 				.filter(dependency -> Objects.equals(artifactDescription.getClassifier(), dependency.getClassifier())
 					&& Objects.equals(artifactDescription.getType(), dependency.getType()))
 				.findFirst();
-			if (!matchingDependency.isPresent()) {
+			if (matchingDependency.isEmpty()) {
 				matchingDependency = firstPass.stream()
 					.findFirst();
-				if (!matchingDependency.isPresent()) {
+				if (matchingDependency.isEmpty()) {
 					throw new MojoExecutionException(String.format(
 						"No version found for artifact %s:%s in project, dependencies, or dependency management",
 						artifactDescription.getGroupId(), artifactDescription.getArtifactId()));