diff --git a/lib/workers/branch.js b/lib/workers/branch.js index 8e75b1830162d6a4605d446f0b63d3686624f48b..691d0c1a27b59761310bd763f01fca3fb5ad93af 100644 --- a/lib/workers/branch.js +++ b/lib/workers/branch.js @@ -210,11 +210,8 @@ async function updateBranch(upgrades) { branch: branchName, }); - const upgradeCount = upgrades.length === 1 - ? '1 upgrade' - : `${upgrades.length} upgrades`; logger.info( - `Branch '${branchName}' has ${upgradeCount}: ${upgrades.map( + `Branch '${branchName}' has ${upgrades.length} upgrade(s): ${upgrades.map( upgrade => upgrade.depName )}` ); diff --git a/lib/workers/package-file.js b/lib/workers/package-file.js index ae1bd98b74982a4f1e9a1dca65f058e434af28a8..630e10c28f9f56e05ab8a1050af5b00c370d8851 100644 --- a/lib/workers/package-file.js +++ b/lib/workers/package-file.js @@ -147,11 +147,8 @@ async function findUpgrades(dependencies) { dep.config ); if (upgrades.length > 0) { - const upgradeCount = upgrades.length === 1 - ? '1 upgrade' - : `${upgrades.length} upgrades`; logger.info( - `Dependency ${dep.depName} has ${upgradeCount} available: ${upgrades.map( + `Dependency ${dep.depName} has ${upgrades.length} upgrade(s) available: ${upgrades.map( upgrade => upgrade.newVersion )}` ); diff --git a/test/api/gitlab.spec.js b/test/api/gitlab.spec.js index 7a2b2538543921d7696d421fea3f2447fcfd3a67..b4fc7a0a40a1b9129e74a18d535f52c9d69fc684 100644 --- a/test/api/gitlab.spec.js +++ b/test/api/gitlab.spec.js @@ -582,12 +582,7 @@ describe('api/gitlab', () => { describe('commitFilesToBranch(branchName, files, message, parentBranch)', () => { it('creates branch', async () => { glGot.mockReturnValueOnce({ statusCode: 404 }); - await gitlab.commitFilesToBranch( - 'some-branch', - [], - 'some-message', - 'parent-branch' - ); + await gitlab.commitFilesToBranch('some-branch', [], 'some-message'); }); it('does not create branch and updates file', async () => { glGot.mockReturnValueOnce({ statusCode: 200 });