-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add a flag to disable local install of artifacts #149
Conversation
Bumps [maven-shade-plugin](https://github.com/apache/maven-shade-plugin) from 2.3 to 3.2.1. - [Release notes](https://github.com/apache/maven-shade-plugin/releases) - [Commits](apache/maven-shade-plugin@maven-shade-plugin-2.3...maven-shade-plugin-3.2.1) Signed-off-by: dependabot[bot] <support@dependabot.com>
Bumps ant from 1.9.2 to 1.10.5. Signed-off-by: dependabot[bot] <support@dependabot.com>
Bumps appengine-api-1.0-sdk from 1.5.5 to 1.9.73. Signed-off-by: dependabot[bot] <support@dependabot.com>
So, I have everything already prepared on my host local repository, I usually run the docker image sharing my host local maven repo to speed things up. On those circumstances installing locally only serves to make the run slower.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @raul-arabaolaza ! An opt-out approach looks good to me
Why do I request a change? I suggest following the same variable pattern as before, hence uppercase. Also, the flag should be reflected in the documentation.
Co-Authored-By: Oleg Nenashev <o.v.nenashev@gmail.com>
I will update the doc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documentation is still missing. The rest LGTM
…-tester into LOCAL_FLAG
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢 🇮🇹
When you are working with snapshots (for example when doing local development) the
run-pct
script tries to install artifacts from the war file to make sure they are available for local runs. However, that could not work when using custom generated war files and is unneeded if you share somehow your local maven repo with all the needed artifacts already there (I use that approach every day)So, this PR introduces a flag to prevent that local installation to happen if desired