[S1161] Add missing @Override to overriding and implementing methods

api/maven-api-cli/src/main/java/org/apache/maven/api/cli/mvn/MavenOptions.java

@@ -229,5 +229,6 @@ public interface MavenOptions extends Options {
      * @return a new MavenOptions instance with interpolated values
      */
     @Nonnull
+    @Override
     MavenOptions interpolate(@Nonnull UnaryOperator<String> callback);
 }

api/maven-api-cli/src/main/java/org/apache/maven/api/cli/mvnenc/EncryptOptions.java

@@ -64,5 +64,6 @@ public interface EncryptOptions extends Options {
      * @return a new EncryptOptions instance with interpolated values
      */
     @Nonnull
+    @Override
     EncryptOptions interpolate(UnaryOperator<String> callback);
 }

api/maven-api-cli/src/main/java/org/apache/maven/api/cli/mvnsh/ShellOptions.java

@@ -39,5 +39,6 @@ public interface ShellOptions extends Options {
      * @return a new EncryptOptions instance with interpolated values
      */
     @Nonnull
+    @Override
     ShellOptions interpolate(UnaryOperator<String> callback);
 }

api/maven-api-core/src/main/java/org/apache/maven/api/ExtensibleEnums.java

@@ -87,6 +87,7 @@ private static class DefaultExtensibleEnum implements ExtensibleEnum {
          *
          * @return the identifier
          */
+        @Override
         public String id() {
             return id;
         }

api/maven-api-core/src/main/java/org/apache/maven/api/Lifecycle.java

@@ -244,6 +244,7 @@ interface PhasePointer extends Pointer {
          *
          * @return the PROJECT pointer type
          */
+        @Override
         default Type type() {
             return Type.PROJECT;
         }
@@ -262,6 +263,7 @@ interface DependenciesPointer extends Pointer {
          *
          * @return the DEPENDENCIES pointer type
          */
+        @Override
         default Type type() {
             return Type.DEPENDENCIES;
         }
@@ -273,6 +275,7 @@ interface ChildrenPointer extends Pointer {
          *
          * @return the CHILDREN pointer type
          */
+        @Override
         default Type type() {
             return Type.CHILDREN;
         }

api/maven-api-core/src/main/java/org/apache/maven/api/Packaging.java

@@ -66,6 +66,7 @@ public interface Packaging extends ExtensibleEnum {
      * The packaging id.
      */
     @Nonnull
+    @Override
     String id();
 
     /**

api/maven-api-core/src/main/java/org/apache/maven/api/Type.java

@@ -134,6 +134,7 @@ public interface Type extends ExtensibleEnum {
      * @return the id of this type, never {@code null}.
      */
     @Nonnull
+    @Override
     String id();
 
     /**

api/maven-api-core/src/main/java/org/apache/maven/api/Version.java

@@ -38,5 +38,6 @@ public interface Version extends Comparable<Version> {
      * {@return the string representation of this version}
      */
     @Nonnull
+    @Override
     String toString();
 }

api/maven-api-core/src/main/java/org/apache/maven/api/VersionConstraint.java

@@ -65,5 +65,6 @@ public interface VersionConstraint {
      * {@return the string representation of this version}
      */
     @Nonnull
+    @Override
     String toString();
 }

api/maven-api-core/src/main/java/org/apache/maven/api/VersionRange.java

@@ -60,6 +60,7 @@ public interface VersionRange {
      * {@return the string representation of this version}
      */
     @Nonnull
+    @Override
     String toString();
 
     /**

api/maven-api-core/src/main/java/org/apache/maven/api/services/ArtifactCoordinatesFactoryRequest.java

@@ -233,6 +233,7 @@ public String getType() {
                 return type;
             }
 
+            @Override
             public String getCoordinatesString() {
                 return coordinatesString;
             }

api/maven-api-core/src/main/java/org/apache/maven/api/services/BaseRequest.java

@@ -45,10 +45,12 @@ protected BaseRequest(@Nonnull S session, RequestTrace trace) {
     }
 
     @Nonnull
+    @Override
     public S getSession() {
         return session;
     }
 
+    @Override
     public RequestTrace getTrace() {
         return trace;
     }

api/maven-api-core/src/main/java/org/apache/maven/api/services/DependencyCoordinatesFactoryRequest.java

@@ -282,6 +282,7 @@ public String getType() {
                 return type;
             }
 
+            @Override
             public String getCoordinatesString() {
                 return coordinateString;
             }

api/maven-api-core/src/main/java/org/apache/maven/api/services/ModelBuilderResult.java

@@ -109,5 +109,6 @@ public interface ModelBuilderResult extends Result<ModelBuilderRequest> {
     /**
      * Creates a human-readable representation of these errors.
      */
+    @Override
     String toString();
 }

api/maven-api-core/src/main/java/org/apache/maven/api/services/xml/XmlReaderRequest.java

@@ -216,6 +216,7 @@ public InputStream getInputStream() {
                 return inputStream;
             }
 
+            @Override
             public Reader getReader() {
                 return reader;
             }

api/maven-api-model/src/main/java/org/apache/maven/api/model/InputLocation.java

@@ -93,6 +93,7 @@ public InputSource getSource() {
         return source;
     }
 
+    @Override
     public InputLocation getLocation(Object key) {
         return locations != null ? locations.get(key) : null;
     }
@@ -108,6 +109,7 @@ public Map<Object, InputLocation> getLocations() {
      * @return InputLocation
      * @since 4.0.0
      */
+    @Override
     public InputLocation getImportedFrom() {
         return importedFrom;
     }

api/maven-api-plugin/src/test/java/org/apache/maven/api/plugin/descriptor/another/ExtendedPluginDescriptorTest.java

@@ -56,6 +56,7 @@ public Builder additionalField(String additionalField) {
                 return this;
             }
 
+            @Override
             public ExtendedPluginDescriptor build() {
                 return new ExtendedPluginDescriptor(this);
             }

compat/maven-artifact/src/main/java/org/apache/maven/artifact/InvalidArtifactRTException.java

@@ -49,6 +49,7 @@ public InvalidArtifactRTException(
         this.baseMessage = message;
     }
 
+    @Override
     public String getMessage() {
         return "For artifact {" + getArtifactKey() + "}: " + getBaseMessage();
     }

compat/maven-artifact/src/main/java/org/apache/maven/artifact/resolver/CyclicDependencyException.java

@@ -32,6 +32,7 @@ public CyclicDependencyException(String message, Artifact artifact) {
         this.artifact = artifact;
     }
 
+    @Override
     public Artifact getArtifact() {
         return artifact;
     }

compat/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/ComparableVersion.java

@@ -285,6 +285,7 @@ public int hashCode() {
             return value.hashCode();
         }
 
+        @Override
         public String toString() {
             return value.toString();
         }
@@ -415,6 +416,7 @@ public int hashCode() {
             return value.hashCode();
         }
 
+        @Override
         public String toString() {
             return value;
         }

compat/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/DefaultArtifactVersion.java

@@ -57,6 +57,7 @@ public boolean equals(Object other) {
         return false;
     }
 
+    @Override
     public int compareTo(ArtifactVersion otherVersion) {
         if (otherVersion instanceof DefaultArtifactVersion defaultArtifactVersion) {
             return this.comparable.compareTo(defaultArtifactVersion.comparable);
@@ -65,26 +66,32 @@ public int compareTo(ArtifactVersion otherVersion) {
         }
     }
 
+    @Override
     public int getMajorVersion() {
         return majorVersion != null ? majorVersion : 0;
     }
 
+    @Override
     public int getMinorVersion() {
         return minorVersion != null ? minorVersion : 0;
     }
 
+    @Override
     public int getIncrementalVersion() {
         return incrementalVersion != null ? incrementalVersion : 0;
     }
 
+    @Override
     public int getBuildNumber() {
         return buildNumber != null ? buildNumber : 0;
     }
 
+    @Override
     public String getQualifier() {
         return qualifier;
     }
 
+    @Override
     public final void parseVersion(String version) {
         comparable = new ComparableVersion(version);
 

compat/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/Restriction.java

@@ -139,6 +139,7 @@ public boolean equals(Object other) {
         }
     }
 
+    @Override
     public String toString() {
         StringBuilder buf = new StringBuilder();
 

compat/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java

@@ -419,6 +419,7 @@ public boolean isSelectedVersionKnown(Artifact artifact) throws OverConstrainedV
         return value;
     }
 
+    @Override
     public String toString() {
         if (recommendedVersion != null) {
             return recommendedVersion.toString();
@@ -465,6 +466,7 @@ public boolean hasRestrictions() {
         return !restrictions.isEmpty() && recommendedVersion == null;
     }
 
+    @Override
     public boolean equals(Object obj) {
         if (this == obj) {
             return true;
@@ -477,6 +479,7 @@ public boolean equals(Object obj) {
         }
     }
 
+    @Override
     public int hashCode() {
         int hash = 7;
         hash = 31 * hash + (recommendedVersion == null ? 0 : recommendedVersion.hashCode());

compat/maven-builder-support/src/main/java/org/apache/maven/building/DefaultProblem.java

@@ -62,18 +62,22 @@ class DefaultProblem implements Problem {
         this.exception = exception;
     }
 
+    @Override
     public String getSource() {
         return source;
     }
 
+    @Override
     public int getLineNumber() {
         return lineNumber;
     }
 
+    @Override
     public int getColumnNumber() {
         return columnNumber;
     }
 
+    @Override
     public String getLocation() {
         StringBuilder buffer = new StringBuilder(256);
 
@@ -101,10 +105,12 @@ public String getLocation() {
         return buffer.toString();
     }
 
+    @Override
     public Exception getException() {
         return exception;
     }
 
+    @Override
     public String getMessage() {
         String msg;
 
@@ -121,6 +127,7 @@ public String getMessage() {
         return msg;
     }
 
+    @Override
     public Severity getSeverity() {
         return severity;
     }

compat/maven-compat/src/main/java/org/apache/maven/DefaultArtifactFilterManager.java

@@ -62,6 +62,7 @@ private synchronized Set<String> getExcludedArtifacts() {
      *
      * @see org.apache.maven.ArtifactFilterManager#getArtifactFilter()
      */
+    @Override
     public ArtifactFilter getArtifactFilter() {
         Set<String> excludes = new LinkedHashSet<>(getExcludedArtifacts());
 
@@ -77,14 +78,17 @@ public ArtifactFilter getArtifactFilter() {
      *
      * @see org.apache.maven.ArtifactFilterManager#getCoreArtifactFilter()
      */
+    @Override
     public ArtifactFilter getCoreArtifactFilter() {
         return new ExclusionSetFilter(getCoreArtifactExcludes());
     }
 
+    @Override
     public void excludeArtifact(String artifactId) {
         getExcludedArtifacts().add(artifactId);
     }
 
+    @Override
     public Set<String> getCoreArtifactExcludes() {
         Set<String> excludes = new LinkedHashSet<>(coreArtifacts);
 

compat/maven-compat/src/main/java/org/apache/maven/DefaultProjectDependenciesResolver.java

@@ -64,11 +64,13 @@ public DefaultProjectDependenciesResolver(
         this.resolutionErrorHandler = resolutionErrorHandler;
     }
 
+    @Override
     public Set<Artifact> resolve(MavenProject project, Collection<String> scopesToResolve, MavenSession session)
             throws ArtifactResolutionException, ArtifactNotFoundException {
         return resolve(Collections.singleton(project), scopesToResolve, session);
     }
 
+    @Override
     public Set<Artifact> resolve(
             MavenProject project,
             Collection<String> scopesToCollect,
@@ -80,12 +82,14 @@ public Set<Artifact> resolve(
                 mavenProjects, scopesToCollect, scopesToResolve, session, getIgnorableArtifacts(mavenProjects));
     }
 
+    @Override
     public Set<Artifact> resolve(
             Collection<? extends MavenProject> projects, Collection<String> scopesToResolve, MavenSession session)
             throws ArtifactResolutionException, ArtifactNotFoundException {
         return resolveImpl(projects, null, scopesToResolve, session, getIgnorableArtifacts(projects));
     }
 
+    @Override
     public Set<Artifact> resolve(
             MavenProject project,
             Collection<String> scopesToCollect,

compat/maven-compat/src/main/java/org/apache/maven/artifact/ArtifactStatus.java

@@ -83,6 +83,7 @@ public static ArtifactStatus valueOf(String status) {
         return retVal != null ? retVal : NONE;
     }
 
+    @Override
     public boolean equals(Object o) {
         if (this == o) {
             return true;
@@ -96,14 +97,17 @@ public boolean equals(Object o) {
         return rank == that.rank;
     }
 
+    @Override
     public int hashCode() {
         return rank;
     }
 
+    @Override
     public String toString() {
         return key;
     }
 
+    @Override
     public int compareTo(ArtifactStatus s) {
         return rank - s.rank;
     }