Skip to content

Commit

Permalink
fix: the merging can not sign the commit (#1235)
Browse files Browse the repository at this point in the history
* fix: the merging can not sign the commit

* docs: add @snowyu as a contributor

Co-authored-by: Jakub T. Jankiewicz <jcubic@onet.pl>
  • Loading branch information
snowyu and jcubic committed Jul 5, 2021
1 parent 1d63afe commit 6af05bd
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/api/merge.js
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,7 @@ export async function merge({
author,
committer,
signingKey,
onSign,
})
} catch (err) {
err.caller = 'git.merge'
Expand Down
3 changes: 3 additions & 0 deletions src/commands/merge.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ import { mergeTree } from '../utils/mergeTree.js'
* @param {number} args.committer.timestamp
* @param {number} args.committer.timezoneOffset
* @param {string} [args.signingKey]
* @param {SignCallback} [args.onSign] - a PGP signing implementation
*
* @returns {Promise<MergeResult>} Resolves to a description of the merge operation
*
Expand All @@ -61,6 +62,7 @@ export async function _merge({
author,
committer,
signingKey,
onSign,
}) {
if (ours === undefined) {
ours = await _currentBranch({ fs, gitdir, fullname: true })
Expand Down Expand Up @@ -145,6 +147,7 @@ export async function _merge({
author,
committer,
signingKey,
onSign,
dryRun,
noUpdateBranch,
})
Expand Down

0 comments on commit 6af05bd

Please sign in to comment.