Skip to content

Commit

Permalink
fix(registry-change): more context in logs
Browse files Browse the repository at this point in the history
  • Loading branch information
Realtin authored and janl committed May 9, 2018
1 parent 852ce6e commit 3d3299e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions jobs/registry-change.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ module.exports = async function (
})

if (!distTag) {
log.info('exited: dependency has no distTag')
log.info(`exited: ${dependency} has no distTag`)
return
}
await npm.put(updatedAt(Object.assign(npmDbDoc, npmDoc)))
Expand All @@ -61,7 +61,7 @@ module.exports = async function (
// so we can heavily optimise by exiting here
// we want to handle different distTags in the future
if (distTag !== 'latest') {
log.info('exited: dependency distTag is not latest')
log.info(`exited: ${dependency} distTag is ${distTag} (not latest)`)
return
}

Expand Down Expand Up @@ -99,7 +99,7 @@ module.exports = async function (
log.info(`exited: no repoDocs found that depend on ${dependency}`)
return
}
log.info(`found ${packageFilesForUpdatedDependency.length} repoDocs that have that dependency`)
log.info(`found ${packageFilesForUpdatedDependency.length} repoDocs that use ${dependency}`)

if (packageFilesForUpdatedDependency.length > 100) statsd.event('popular_package')

Expand Down

0 comments on commit 3d3299e

Please sign in to comment.