fix: linting, update dependencies and github api package name

Author: jeremenichelli

package.json

@@ -38,7 +38,8 @@
     "deploy": "gh-pages -d build"
   },
   "devDependencies": {
-    "alex": "^4.1.0",
+    "@octokit/rest": "^15.2.6",
+    "alex": "^5.1.0",
     "async": "^2.5.0",
     "autoprefixer": "^7.2.3",
     "babel-core": "^6.26.0",
@@ -62,7 +63,6 @@
     "fontgen-loader": "^0.2.1",
     "front-matter": "^2.3.0",
     "gh-pages": "^1.0.0",
-    "github": "^10.0.0",
     "html-webpack-plugin": "^2.30.1",
     "html-webpack-template": "^6.1.0",
     "http-server": "^0.10.0",

src/scripts/fetch.sh

@@ -13,7 +13,7 @@ set -e # Exit with nonzero exit code if anything fails
 # node ./src/scripts/fetch_package_names.js "webpack-contrib" "-loader" | node ./src/scripts/fetch_package_files.js "README.md" "./generated/loaders"
 # node ./src/scripts/fetch_package_names.js "babel" "babel-loader" | node ./src/scripts/fetch_package_files.js "README.md" "./generated/loaders"
 # node ./src/scripts/fetch_package_names.js "postcss" "postcss-loader" | node ./src/scripts/fetch_package_files.js "README.md" "./generated/loaders"
-# node ./src/scripts/fetch_package_names.js "peerigon" "extract-loader" | node ./src/scripts/fetch_package_files.js "README.md" "./generated/loaders"
+ node ./src/scripts/fetch_package_names.js "peerigon" "extract-loader" | node ./src/scripts/fetch_package_files.js "README.md" "./generated/loaders"
 
 # Fetch webpack-contrib (and various other) plugin repositories
 # node ./src/scripts/fetch_package_names.js "webpack-contrib" "-webpack-plugin" | node ./src/scripts/fetch_package_files.js "README.md" "./generated/plugins"

src/scripts/fetch_package_names.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 // ./fetch_package_names <suffix> > output
 // ./fetch_package_names "-loader" > output.json
-const GitHubApi = require("github");
+const GitHubApi = require('@octokit/rest');
 
 if (require.main === module) {
     main();
@@ -39,13 +39,13 @@ function fetchPackageNames(options, cb) {
   github.repos.getForOrg({
     org: options.organization,
     per_page: 100
-  }, function (err, d) {
+  }, function (err, response) {
     if (err) {
       return cb(err);
     }
 
-    return cb(null, d.data.filter(function(o) {
-      return o.name.endsWith(options.suffix);
+    return cb(null, response.data.filter(function(organization) {
+      return organization.name.endsWith(options.suffix);
     }));
   });
 }

src/scripts/fetch_supporters.js

@@ -3,7 +3,7 @@ const fs = require('fs');
 const request = require('request');
 
 const REQUIRED_KEYS = ['totalDonations', 'id'];
-const filename = '_supporters.json'
+const filename = '_supporters.json';
 
 request('https://opencollective.com/api/groups/webpack/backers', (err, response, body) => {
   if (err) console.error('Failed to fetch backers: ', err);

webpack.common.js

@@ -114,14 +114,14 @@ module.exports = (env = {}) => ({
             .replace(item.extension, '')
             .replace(options.dir, '')
             .replace(/\/index$/, '')
-            .replace(/^$/, '/')
+            .replace(/^$/, '/');
 
         // TODO: Strip `_` prefix from filenames in `url`
         if (item.type === 'file') {
-          let content = fs.readFileSync(item.path, 'utf8')
-          let { attributes } = FrontMatter(content)
-          Object.assign(item, attributes)
-          item.anchors = [] // TODO: Add actual anchors
+          let content = fs.readFileSync(item.path, 'utf8');
+          let { attributes } = FrontMatter(content);
+          Object.assign(item, attributes);
+          item.anchors = []; // TODO: Add actual anchors
 
         } else {
           // TODO: Add directory (section) attributes and index url (if necessary)
@@ -137,4 +137,4 @@ module.exports = (env = {}) => ({
     publicPath: '/',
     filename: '[name].bundle.js'
   }
-})
+});

webpack.dev.js

@@ -32,4 +32,4 @@ module.exports = env => merge(common(env), {
     compress: true,
     historyApiFallback: true
   }
-})
+});

webpack.prod.js

@@ -17,7 +17,7 @@ const prod = {
       'process.env.NODE_ENV': JSON.stringify('production')
     })
   ]
-}
+};
 
 // Export both SSG and SPA configurations
 module.exports = env => [
@@ -81,4 +81,4 @@ module.exports = env => [
       })
     ]
   })
-]
+];