-
Notifications
You must be signed in to change notification settings - Fork 22
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
Drop the cruft #66
Drop the cruft #66
Conversation
Changes: * drop MAT * simplify all the hoops and loops * stop propagating bad practice (passphrase in files on SCM)
@@ -177,7 +188,10 @@ AbstractGpgSigner newSigner(MavenProject project) throws MojoExecutionException, | |||
signer.setLockMode(lockMode); | |||
signer.setArgs(gpgArguments); | |||
|
|||
loadGpgPassphrase(); | |||
String passphrase = System.getenv(MAVEN_GPG_PASSPHRASE); |
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.
Shouldn’t the env variable be configurable with the current value as a default ?
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.
It could be, yes, even better
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.
Fixed
splitting at least the MAT migration vs the gpg passphrase change would ease review and learning |
Once @olamy or @slawekjaranowski figure out this PR apache/maven-gh-actions-shared#96 (ideally as "v4"), will split this PR into several distinct PRs |
This PR is going to be "chopped" into a series of smaller PRs, along the lines for 3.2.0 version: |
Continued along the lines of #65
Drop even more cruft
Changes: