Skip to content

Commit

Permalink
fix: only parse commit messages during git node backport analysis (#651)
Browse files Browse the repository at this point in the history
Previously we parse the entire commit, including the diff, which
can result in ENOBUFS errors. Adding `-s` option to the `git show`
command would eliminate the commit body in the output, which
we don't need, so the error can be less likely to happen during
commit message analysis.
  • Loading branch information
joyeecheung committed Oct 12, 2022
1 parent c4ab7f5 commit 4e59a64
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/backport_session.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export default class BackportSession extends Session {

getCommitMessage(rev) {
return runSync('git',
['show', '--format=%B', rev]
['show', '--format=%B', '-s', rev]
).trim();
}

Expand Down

0 comments on commit 4e59a64

Please sign in to comment.