From f0642f98ff7ff2f95622012554830ebecd399c53 Mon Sep 17 00:00:00 2001 From: Kristjan Broder Lund Date: Thu, 28 Mar 2019 19:27:09 +0000 Subject: [PATCH] fix: stop suggesting npm publish if package.json was not updated --- lib/lifecycles/tag.js | 3 ++- test.js | 7 +++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/lib/lifecycles/tag.js b/lib/lifecycles/tag.js index 9de7cd978..eb3255760 100644 --- a/lib/lifecycles/tag.js +++ b/lib/lifecycles/tag.js @@ -1,3 +1,4 @@ +const bump = require('../lifecycles/bump') const chalk = require('chalk') const checkpoint = require('../checkpoint') const figures = require('figures') @@ -28,7 +29,7 @@ function execTag (newVersion, pkgPrivate, args) { .then(() => runExec(args, 'git rev-parse --abbrev-ref HEAD')) .then((currentBranch) => { let message = 'git push --follow-tags origin ' + currentBranch.trim() - if (pkgPrivate !== true) { + if (pkgPrivate !== true && bump.getUpdatedConfigs()['package.json']) { message += ' && npm publish' if (args.prerelease !== undefined) { if (args.prerelease === '') { diff --git a/test.js b/test.js index d0ca25e55..d9cc33bad 100644 --- a/test.js +++ b/test.js @@ -1030,6 +1030,13 @@ describe('standard-version', function () { output.stdout.should.include('v5.1.0') }) }) + + it('does not display `npm publish` if there is no package.json', function () { + shell.rm('package.json') + const result = execCli() + result.code.should.equal(0) + result.stdout.should.not.match(/npm publish/) + }) }) describe('configuration', () => {