-
Notifications
You must be signed in to change notification settings - Fork 69
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
PR #8
Comments
It's almost ok for me, but I think better to use $HOME or $USER/home or another way to detect home folder for current user without hardcoding user or entire path. Although I have not checked existence of $ HOME or $ USER variables in github environments. |
I did some experiments yesterday and there is some stuff that is shared.
From all them I see only So if we could have As I am no maven expert I dunno if this is applicable. But it would also open the possibility to push a settings.xml in it! On a second note, this might make no sense as the workspace assumingly exists per workspace not per "runner". Hmmm, how can one level1 cache maven on a runner? |
@stCarolas we can also do in your action.yml:
with examples:
This example would make the m2 folder available for docker actions! |
@mambax parameter in action.yml is good for me. Also I would prefer default value as in setup-java - so another actions will use same maven home by default. |
I got in touch with GHES Support. I provided them a demo Now in there you see: I did not find a suitable solution yet as there is now way currently to handle volumes to docker actions (Except using |
@dominikmeyersap what the point of running maven on agent and maven in docker simultaneously? Is the problem in mounting maven cache to container ? |
Correct 💯 I have a gut feeling that with GHES 3 (which we get soon) we could use |
I would like to make PR, to open up the .m2 home path for configuration.
Are you accepting PRs or is it in maintenance mode? 😊
setup-maven/src/installer.ts
Line 16 in d55bf2a
this I would like to override with
/github/home
, does it even make sense?The text was updated successfully, but these errors were encountered: