From e4deb4c261dedab0dc294e81d24f15050f012636 Mon Sep 17 00:00:00 2001 From: Andrea Torlaschi Date: Sun, 12 Jul 2020 18:32:17 +0200 Subject: [PATCH 1/2] fix(commit): don't enclose message in quotes --- lib/lifecycles/commit.js | 2 +- test.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/lifecycles/commit.js b/lib/lifecycles/commit.js index a4bdb7909..bcde6f28a 100644 --- a/lib/lifecycles/commit.js +++ b/lib/lifecycles/commit.js @@ -67,7 +67,7 @@ function execCommit (args, newVersion) { args.commitAll ? [] : toAdd, [ '-m', - `"${formatCommitMessage(args.releaseCommitMessageFormat, newVersion)}"` + `${formatCommitMessage(args.releaseCommitMessageFormat, newVersion)}` ] ) ) diff --git a/test.js b/test.js index 5b011fdba..a2162feb4 100644 --- a/test.js +++ b/test.js @@ -263,7 +263,7 @@ describe('cli', function () { return line ? JSON.parse(line) : line }) /* eslint-disable no-useless-escape */ - captured[captured.length - 4].should.deep.equal(['commit', '-S', 'CHANGELOG.md', 'package.json', '-m', '\"chore(release): 1.0.1\"']) + captured[captured.length - 4].should.deep.equal(['commit', '-S', 'CHANGELOG.md', 'package.json', '-m', 'chore(release): 1.0.1']) captured[captured.length - 3].should.deep.equal(['tag', '-s', 'v1.0.1', '-m', '\"chore(release): 1.0.1\"']) /* eslint-enable no-useless-escape */ unmock() From d895b5687e29523be7b6549a691977a987c88fdb Mon Sep 17 00:00:00 2001 From: Andrea Torlaschi Date: Sun, 12 Jul 2020 18:37:48 +0200 Subject: [PATCH 2/2] fix(tag): don't enclose name in quotes --- lib/lifecycles/tag.js | 2 +- test.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/lifecycles/tag.js b/lib/lifecycles/tag.js index 214e69c92..f160ce743 100644 --- a/lib/lifecycles/tag.js +++ b/lib/lifecycles/tag.js @@ -25,7 +25,7 @@ function execTag (newVersion, pkgPrivate, args) { tagOption = '-a' } checkpoint(args, 'tagging release %s%s', [args.tagPrefix, newVersion]) - return runExecFile(args, 'git', ['tag', tagOption, args.tagPrefix + newVersion, '-m', `"${formatCommitMessage(args.releaseCommitMessageFormat, newVersion)}"`]) + return runExecFile(args, 'git', ['tag', tagOption, args.tagPrefix + newVersion, '-m', `${formatCommitMessage(args.releaseCommitMessageFormat, newVersion)}`]) .then(() => runExecFile('', 'git', ['rev-parse', '--abbrev-ref', 'HEAD'])) .then((currentBranch) => { let message = 'git push --follow-tags origin ' + currentBranch.trim() diff --git a/test.js b/test.js index a2162feb4..83627fa66 100644 --- a/test.js +++ b/test.js @@ -264,7 +264,7 @@ describe('cli', function () { }) /* eslint-disable no-useless-escape */ captured[captured.length - 4].should.deep.equal(['commit', '-S', 'CHANGELOG.md', 'package.json', '-m', 'chore(release): 1.0.1']) - captured[captured.length - 3].should.deep.equal(['tag', '-s', 'v1.0.1', '-m', '\"chore(release): 1.0.1\"']) + captured[captured.length - 3].should.deep.equal(['tag', '-s', 'v1.0.1', '-m', 'chore(release): 1.0.1']) /* eslint-enable no-useless-escape */ unmock() })