From 94cb3523956c5a1a0422c48e18e0d37ab3f0d5f4 Mon Sep 17 00:00:00 2001 From: Feross Aboukhadijeh Date: Thu, 8 Feb 2018 03:55:01 -0800 Subject: [PATCH] Ignore packages that are missing a "maintainers" field Fixes #4 --- cmd.js | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/cmd.js b/cmd.js index 4c80061..deabf10 100755 --- a/cmd.js +++ b/cmd.js @@ -239,6 +239,13 @@ function computeAuthorsPkgNames (pkgs, pkgDownloads, directPkgNames) { const authorPkgNames = {} pkgs.forEach(pkg => { + if (!pkg.maintainers) { + // Ignore packages that are missing a "maintainers" field (e.g. + // http://registry.npmjs.com/vargs/latest). This appears to happen on very old + // packages. My guess is that the "maintainers" field only started getting + // added to release metadata recently. + return + } pkg.maintainers .map(maintainer => maintainer.name) .forEach(author => {