-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Signed release #798
Signed release #798
Conversation
@wolviecb |
d5a337c
to
8cd30fb
Compare
Changes made, commits squashed, thx for the feedback :) |
bin/git-release
Outdated
@@ -34,6 +34,17 @@ if test $# -gt 0; then | |||
-c) need_changelog=true;; | |||
-r) remote=$2; shift ;; | |||
-m) msg=$2; shift ;; | |||
-s) | |||
test -n "$keyid" && |
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.
A little cavil: please indent the -s)
block like the --semver)
one below.
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.
done
bin/git-release
Outdated
sign=true | ||
;; | ||
-u) | ||
test -n "$sign" && |
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.
Ditto
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.
done
8cd30fb
to
704b1b4
Compare
@wolviecb |
This PR adds support for gpg signed tags for the
git-release
commandgit release -s 1
Will generate a tag
1
signed by the default gpg keygit release -u key-id 1
Will generate a tag
1
signed bykey-id