Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MNG-8084] Change six remaining imports of ModelVersionParser. #1527

Merged
merged 1 commit into from
May 21, 2024

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
13 checks passed
@gnodet gnodet added this to the 4.0.0-beta-1 milestone May 21, 2024
This pull request was closed.
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.

3 participants