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] add di & api-impl #1461

Merged
merged 1 commit into from
Apr 24, 2024
Merged

[MNG-8084] add di & api-impl #1461

merged 1 commit into from
Apr 24, 2024

Conversation

hboutemy
Copy link
Member

@hboutemy hboutemy commented Mar 31, 2024

@hboutemy
Copy link
Member Author

notice that there is a chicken and egg problem between resolver and Maven: api-impl depends on resolver supplier, which depends on resolver-provider

@cstamas
Copy link
Member

cstamas commented Mar 31, 2024

Supplier is to be moved to api-impl, as it is not meant to be used in Maven (only in projects that "run maven without maven", ie. resolver integrations w/ maven models). The plan is to move supplier to api-impl, and have resolver supplier depends on api-impl instead. @gnodet

@cstamas
Copy link
Member

cstamas commented Apr 2, 2024

See #1460

@gnodet
Copy link
Contributor

gnodet commented Apr 5, 2024

Fwiw, this will be outdated with #1465 and later #1457

@gnodet
Copy link
Contributor

gnodet commented Apr 12, 2024

Fwiw, this will be outdated with #1465 and later #1457

Also maven-compat is not used anymore (but only included in the distribution for compatibility).

@gnodet
Copy link
Contributor

gnodet commented Apr 12, 2024

maven-4 0 0-alpha-14-SNAPSHOT

It's not readable unfortunately, but it started from generated graph, so it's correct with current master.

@gnodet
Copy link
Contributor

gnodet commented Apr 12, 2024

maven-4 0 0-alpha-14-SNAPSHOT

It's not readable unfortunately, but it started from generated graph, so it's correct with current master.

Note that the ultimate goal would be, imho, to have a maven-compat3 with all the things from maven 3, and have maven-core not depend on the v3 artifacts / api anymore.

drop model-transform deleted in MNG-7877
@hboutemy
Copy link
Member Author

PR updated https://maven.apache.org/ref/4-LATEST/
objective of this PR is to show what we have today, that will help us define targets: schema will evolve when useful

@hboutemy hboutemy added this to the 4.0.0-beta-1 milestone Apr 23, 2024
Copy link
Contributor

@gnodet gnodet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good enough to me, even though it's not perfectly inline with current status.

@hboutemy
Copy link
Member Author

hboutemy commented Apr 24, 2024

I'm happy to update it if you tell me what remains to do: it's just a small update of an OpenOffice document, save as .svg and run the update prepare-svg.sh script

It's not readable unfortunately, but it started from generated graph, so it's correct with current master.

BTW, I'd be interested to be able to create this type of automatic drawing as a reference

@gnodet
Copy link
Contributor

gnodet commented Apr 24, 2024

I'm happy to update it if you tell me what remains to do: it's just a small update of an OpenOffice document, save as .svg and run the update prepare-svg.sh script

It's not readable unfortunately, but it started from generated graph, so it's correct with current master.

BTW, I'd be interested to be able to create this type of automatic drawing as a reference

mvn org.fusesource.mvnplugins:maven-graph-plugin:reactor does render something automatically...
reactor-graph

@gnodet
Copy link
Contributor

gnodet commented Apr 24, 2024

I'm happy to update it if you tell me what remains to do: it's just a small update of an OpenOffice document, save as .svg and run the update prepare-svg.sh script

It's not readable unfortunately, but it started from generated graph, so it's correct with current master.

BTW, I'd be interested to be able to create this type of automatic drawing as a reference

mvn org.fusesource.mvnplugins:maven-graph-plugin:reactor -Dhide-version -Dhide-group-id -Dhide-scope=test -Dhide-transitive gives the following:
reactor-graph

@hboutemy hboutemy merged commit 41d1950 into master Apr 24, 2024
23 of 28 checks passed
@hboutemy hboutemy deleted the MNG-8084-doc branch April 24, 2024 16:01
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