Skip to content

Conversation

@desruisseaux
Copy link
Contributor

This commit aims to complete d075fe7, which moved ModelVersionParser interface from the org.apache.maven.model.version package to org.apache.maven.api.services.model. Completing this change is necessary for building maven-plugin-tools or maven-plugin-testing.

This commit deletes the old interface for making sure that there is no accidental use. The old and new interfaces were identical except for the package names and the imports. As a result of this change, the following implementation classes are identical except for package names and imports:

  • maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelVersionParser.java
  • maven-api-impl/src/main/java/org/apache/maven/internal/impl/DefaultModelVersionParser.java

This pull request does not resolve the duplication in above implementation classes.

This commit aims to complete d075fe7, which moved `ModelVersionParser`
interface from the `org.apache.maven.model.version` package to `org.apache.maven.api.services.model`.
Completing this change is necessary for building `maven-plugin-tools` or `maven-plugin-testing`.

This commit deletes the old interface for making sure that there is no accidental use.
The old and new interfaces were identical except for the package names and the imports.
As a result of this change, the following implementation classes are identical except
for package names and imports:

* `maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/DefaultModelVersionParser.java`
* `maven-api-impl/src/main/java/org/apache/maven/internal/impl/DefaultModelVersionParser.java`
@gnodet gnodet merged commit 39696f2 into apache:master May 21, 2024
@gnodet gnodet added this to the 4.0.0-beta-1 milestone May 21, 2024
@jira-importer
Copy link

Resolve #10798

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.

4 participants