Skip to content

Commit 9ef4db2

Browse files
committed
Add missing Nullable annotations
1 parent 624ea84 commit 9ef4db2

File tree

3 files changed

+11
-2
lines changed

3 files changed

+11
-2
lines changed

src/main/java/org/gradlex/javamodule/moduleinfo/ModuleInfo.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
package org.gradlex.javamodule.moduleinfo;
1818

1919
import org.gradle.api.model.ObjectFactory;
20+
import org.jspecify.annotations.Nullable;
2021

2122
import java.util.Arrays;
2223
import java.util.LinkedHashMap;
@@ -30,6 +31,7 @@
3031
@SuppressWarnings("unused")
3132
public class ModuleInfo extends ModuleSpec {
3233

34+
@Nullable
3335
private final String moduleVersion;
3436

3537
boolean openModule = true;
@@ -47,7 +49,7 @@ public class ModuleInfo extends ModuleSpec {
4749
boolean patchRealModule;
4850
boolean preserveExisting;
4951

50-
ModuleInfo(String identifier, String moduleName, String moduleVersion, ObjectFactory objectFactory) {
52+
ModuleInfo(String identifier, String moduleName, @Nullable String moduleVersion, ObjectFactory objectFactory) {
5153
super(identifier, moduleName);
5254
this.moduleVersion = moduleVersion;
5355
}
@@ -124,6 +126,7 @@ public void ignoreServiceProvider(String provider, String... implementations) {
124126
/**
125127
* @return configured version of the Module
126128
*/
129+
@Nullable
127130
public String getModuleVersion() {
128131
return moduleVersion;
129132
}

src/main/java/org/gradlex/javamodule/moduleinfo/ModuleSpec.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
import org.gradle.api.artifacts.MinimalExternalModuleDependency;
2020
import org.gradle.api.provider.Provider;
21+
import org.jspecify.annotations.Nullable;
2122

2223
import java.io.Serializable;
2324
import java.util.ArrayList;
@@ -33,6 +34,7 @@
3334
public abstract class ModuleSpec implements Serializable {
3435

3536
private final String identifier;
37+
@Nullable
3638
private final String classifier; // optional
3739
private final String moduleName;
3840
private final List<String> removedPackages = new ArrayList<>();
@@ -63,6 +65,7 @@ public String getIdentifier() {
6365
/**
6466
* @return classifier, as an addition to group:name coordinates, if defined
6567
*/
68+
@Nullable
6669
public String getClassifier() {
6770
return classifier;
6871
}

src/main/java/org/gradlex/javamodule/moduleinfo/PublishedMetadata.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
import org.gradle.api.tasks.SourceSet;
3434
import org.gradle.api.tasks.SourceSetContainer;
3535
import org.gradle.util.GradleVersion;
36+
import org.jspecify.annotations.Nullable;
3637

3738
import java.io.Serializable;
3839
import java.util.ArrayList;
@@ -54,6 +55,7 @@ public class PublishedMetadata implements Serializable {
5455
private final List<String> requires = new ArrayList<>();
5556
private final List<String> requiresTransitive = new ArrayList<>();
5657
private final List<String> requiresStaticTransitive = new ArrayList<>();
58+
@Nullable
5759
private String errorMessage = null;
5860

5961
PublishedMetadata(String gav, Project project, ExtraJavaModuleInfoPluginExtension extension) {
@@ -95,7 +97,7 @@ private List<String> componentVariant(Provider<String> versionsProvidingConfigur
9597
return firstAndOnlyComponentDependencies(singleComponentVariantResolver);
9698
}
9799

98-
private Configuration maybeCreateDefaultVersionSourcConfiguration(ConfigurationContainer configurations, ObjectFactory objects, SourceSetContainer sourceSets) {
100+
private Configuration maybeCreateDefaultVersionSourcConfiguration(ConfigurationContainer configurations, ObjectFactory objects, @Nullable SourceSetContainer sourceSets) {
99101
String name = DEFAULT_VERSION_SOURCE_CONFIGURATION;
100102
Configuration existing = configurations.findByName(name);
101103
if (existing != null) {
@@ -174,6 +176,7 @@ public List<String> getRequiresStaticTransitive() {
174176
return requiresStaticTransitive;
175177
}
176178

179+
@Nullable
177180
public String getErrorMessage() {
178181
return errorMessage;
179182
}

0 commit comments

Comments
 (0)