-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
gitconfig gpgsign = true
should only sign signed-off-by commits
#64
Comments
Thanks for the report! I don't regularly use the I do see that Alternatively, you can also consider using the recently added I'm not currently seeing a better workaround for this, but let me know your thoughts! |
Thanks for your reply. The problem with having a static succession is that the author tag never changes. We used git-duet specifically to prevent this. Another workaround would be to disable the |
Aha, I see, yes. A couple of ideas:
Open to other implementation approaches! |
I am having the same issue as @thardeck , however I wanted to use the
With this setup, I also noticed another strange behavior:
So it appears that the committer gets set, but not unset. Is this behavior intended? I'm happy to test more with my setup. |
Apologies for the delayed response! What you are seeing is correct behavior,
You can run Example:
|
Hey @jszwedko no problem - thanks for the reply! Here is a session illustrating the behavior i believe to be incorrect:
Using |
Aha! I see. This seems very similar to #81 .
|
By default all my my git commits are signed by gpg. I have added the following snippet to my
.gitconfig
:The issue is when using git-duet all commits are signed, but then Github shows the one which are committed by me, but signed-off by my colleague as unverified because he does not have my gpg key in his github settings and he should not have.
So I suppose the right approach would be to only sign commits which are "Signed-off-by" by the user mentioned in the
.gitconfig
.Or github changes their behavior and also verifies the commits signed-off by somebody else with the gpg key of the committer.
The text was updated successfully, but these errors were encountered: