-
Notifications
You must be signed in to change notification settings - Fork 10
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
ci: Add release automation #28
Conversation
6d3bc49
to
5b517db
Compare
6efc25c
to
fc5d54c
Compare
b83795c
to
277a725
Compare
277a725
to
4d123d6
Compare
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.
LGTM
Actually, @coderbirju can you also verify that the packaged binary |
Updated the release-automation to install go-licenses globally.
The |
I might have missed it but I don't see it's happening. I am referring the the validation of the output of |
Oh just realized that we currently do not have that option. But we need to verify it's |
@coderbirju #40 could be useful |
9cdb2eb
to
513dfe4
Compare
1bf990f
to
9574265
Compare
9574265
to
e9ee71b
Compare
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.
LGTM
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
e9ee71b
to
ee9a34e
Compare
needs #40 #42
Issue #, if available:
Description of changes:
Testing done:
License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.