Skip to content

git: Sign revup-created commits when commit.gpgSign is set#264

Merged
jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/gpg
Jun 26, 2026
Merged

git: Sign revup-created commits when commit.gpgSign is set#264
jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/gpg

Conversation

@jerry-skydio

Copy link
Copy Markdown
Collaborator

commit-tree (plumbing) ignores commit.gpgSign, so read it from git
config and pass -S explicitly. Read in the existing make_git async
gather alongside the other config lookups.

Co-Authored-By: Anurag Makineni anurag.makineni@wayve.ai

@jerry-skydio

Copy link
Copy Markdown
Collaborator Author

Reviews in this chain:
#264 git: Sign revup-created commits when commit.gpgSign is set

@jerry-skydio

jerry-skydio commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator Author
# head base diff date summary
0 b9f545a3 e03937b2 diff Jun 19 14:19 PM 1 file changed, 14 insertions(+)
1 21e4b07b e03937b2 diff Jun 19 14:22 PM 1 file changed, 4 insertions(+), 2 deletions(-)
2 de8b12af e03937b2 diff Jun 22 12:57 PM 1 file changed, 14 insertions(+), 9 deletions(-)

@anurag-makineni-wayve anurag-makineni-wayve left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great, thanks @jerry-skydio!

commit-tree (plumbing) ignores commit.gpgSign, so read it from git
config and pass -S explicitly. Read in the existing make_git async
gather alongside the other config lookups.

Co-Authored-By: Anurag Makineni <anurag.makineni@wayve.ai>
@jerry-skydio jerry-skydio force-pushed the jerry/revup/main/gpg branch from 21e4b07 to de8b12a Compare June 22, 2026 19:57
@jerry-skydio jerry-skydio merged commit 9212256 into main Jun 26, 2026
5 checks passed
@jerry-skydio jerry-skydio deleted the jerry/revup/main/gpg branch June 26, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants