diff --git a/commands/bootstrap/index.js b/commands/bootstrap/index.js index 360fb03dca8..a1daeb380bb 100644 --- a/commands/bootstrap/index.js +++ b/commands/bootstrap/index.js @@ -135,26 +135,6 @@ class BootstrapCommand extends Command { let chain = Promise.resolve(); chain = chain.then(() => { - if (this.options.scope) { - this.logger.notice("filter", "including %j", this.options.scope); - } - - if (this.options.ignore) { - this.logger.notice("filter", "excluding %j", this.options.ignore); - } - - if (this.options.since) { - this.logger.notice("filter", "changed since %j", this.options.since); - } - - if (this.options.includeFilteredDependents) { - this.logger.notice("filter", "including filtered dependents"); - } - - if (this.options.includeFilteredDependencies) { - this.logger.notice("filter", "including filtered dependencies"); - } - return getFilteredPackages(this.targetGraph, this.execOpts, this.options); }); diff --git a/core/filter-options/lib/get-filtered-packages.js b/core/filter-options/lib/get-filtered-packages.js index 79bd3c0579e..954f9d1100e 100644 --- a/core/filter-options/lib/get-filtered-packages.js +++ b/core/filter-options/lib/get-filtered-packages.js @@ -1,11 +1,20 @@ "use strict"; +const log = require("npmlog"); const collectUpdates = require("@lerna/collect-updates"); const filterPackages = require("@lerna/filter-packages"); module.exports = getFilteredPackages; function getFilteredPackages(packageGraph, execOpts, options) { + if (options.scope) { + log.notice("filter", "including %j", options.scope); + } + + if (options.ignore) { + log.notice("filter", "excluding %j", options.ignore); + } + let chain = Promise.resolve(); chain = chain.then(() => @@ -13,6 +22,8 @@ function getFilteredPackages(packageGraph, execOpts, options) { ); if (options.since !== undefined) { + log.notice("filter", "changed since %j", options.since); + chain = chain.then(filteredPackages => Promise.resolve(collectUpdates(filteredPackages, packageGraph, execOpts, options)).then(updates => { const updated = new Set(updates.map(({ pkg }) => pkg.name)); @@ -23,10 +34,14 @@ function getFilteredPackages(packageGraph, execOpts, options) { } if (options.includeFilteredDependents) { + log.notice("filter", "including filtered dependents"); + chain = chain.then(filteredPackages => packageGraph.addDependents(filteredPackages)); } if (options.includeFilteredDependencies) { + log.notice("filter", "including filtered dependencies"); + chain = chain.then(filteredPackages => packageGraph.addDependencies(filteredPackages)); } diff --git a/core/filter-options/package.json b/core/filter-options/package.json index be192113367..37758f3940b 100644 --- a/core/filter-options/package.json +++ b/core/filter-options/package.json @@ -34,6 +34,7 @@ "dependencies": { "@lerna/collect-updates": "file:../../utils/collect-updates", "@lerna/filter-packages": "file:../../utils/filter-packages", - "dedent": "^0.7.0" + "dedent": "^0.7.0", + "npmlog": "^4.1.2" } } diff --git a/package-lock.json b/package-lock.json index 938bbac640e..1b1a70ba333 100644 --- a/package-lock.json +++ b/package-lock.json @@ -974,7 +974,8 @@ "requires": { "@lerna/collect-updates": "file:utils/collect-updates", "@lerna/filter-packages": "file:utils/filter-packages", - "dedent": "^0.7.0" + "dedent": "^0.7.0", + "npmlog": "^4.1.2" } }, "@lerna/filter-packages": {