Skip to content

Conversation

@kressi
Copy link

@kressi kressi commented Nov 14, 2025

Since not everyone has/wants Maven locally installed, I'm proposing to add Maven wrapper to the project. Additionally, this ensures the project is built with the desired Maven version.

@pivovarit
Copy link
Member

I'm all for it, but one question remains - will this make GitHub Actions install Maven on every build first? 🤔

@kressi
Copy link
Author

kressi commented Dec 15, 2025

Yes, only .m2/repository is being cached. It does not look like cache can easily be extended with additional paths. I think setting up cache for .m2/wrapper/dist will complicate things unnecessarily. Is it ok if I add wrapper as type=bin, so it's part of the repository?

@pivovarit
Copy link
Member

Yes, let's try

@kressi
Copy link
Author

kressi commented Dec 15, 2025

Pushed with -Dtype=bin

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