Skip to content

Cleanup repositories #18

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

Closed

Conversation

ascheman
Copy link
Contributor

Some projects are (already) archived or even no longer used by the Apache team.
Therefore, they are not needed when checking out the Maven sources.
Instead we add https://github.com/codehaus-plexus/.github as part of the sources (as it contains common GitHub actions for Plexus components).

This change is a late result of support-and-care/maven-support-and-care#77.

@hboutemy
Copy link
Member

hboutemy commented May 24, 2025

I tried to do a review: each commit added so much to discuss that I don't want to start reviewing the full PR

please create one PR per change (that should even be more granular than the 3 commits), because each change nature needs some discussion: many small discussions on many small changes will be much easier than one global one

ascheman added 3 commits May 24, 2025 09:43
as it contains shared GitHub action workflow components
In particular some more components are archived ...
@ascheman
Copy link
Contributor Author

I tried to do a review: each commit added so much to discuss that I don't want to start reviewing the full PR

please create one PR per change (that should even be more granular than the 3 commits), because each change nature needs some discussion: many small discussions on many small changes will be much easier than one global one

Thanks for the feedback, I have now raised #21, #22, and #23 to replace this mostly.
I would like to see a little bit more alphabetical ordering in some files (to make alignment in the future easier), but postpone this to a later point in time.

@ascheman ascheman closed this May 25, 2025
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.

2 participants