-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-25376 [create-release] Fix double .asc #2755
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
Conversation
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
apurtell
left a comment
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.
99% of the changes are out of scope for the fix?
|
Maybe do the one liner here and the misc cleanups on another? |
+1 |
d05c593 to
d419cf1
Compare
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
bq. 99% of the changes are out of scope for the fix? np |
d419cf1 to
ab7ee16
Compare
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
Ah and that's how my manual invocation messed up too, I did "mvn clean install deploy" (sigh) |
Don't install when we call deploy -- makes for doubling of .asc.
Added more to the README while in here and added logging
timestamp so we can tell where we are spending time.