Clean code

packages/strapi-generate-plugin/json/package.json.js

@@ -55,8 +55,8 @@ module.exports = scope => {
       'url': scope.website || ''
     }],
     'engines': {
-      'node': '>= 7.0.0',
-      'npm': '>= 3.0.0'
+      'node': '>= 9.0.0',
+      'npm': '>= 5.3.0'
     },
     'license': scope.license || 'MIT'
   });

packages/strapi-plugin-content-manager/config/layout.js

@@ -1 +0,0 @@
-module.exports = {};

packages/strapi-plugin-graphql/package.json

@@ -46,8 +46,8 @@
     }
   ],
   "engines": {
-    "node": ">= 7.0.0",
-    "npm": ">= 3.0.0"
+    "node": ">= 9.0.0",
+    "npm": ">= 5.3.0"
   },
   "license": "MIT"
 }

packages/strapi-plugin-graphql/services/GraphQL.js

@@ -509,8 +509,6 @@ module.exports = {
           default:
         }
 
-        // TODO:
-        // - Handle limit, skip, etc options
         _.merge(acc.resolver[globalId], {
           [association.alias]: async (obj, options, context) => {
             // Construct parameters object to retrieve the correct related entries.

packages/strapi/lib/hooks/index.js

@@ -84,7 +84,7 @@ module.exports = async function() {
           }
 
           // Initialize array.
-          let previousDependencies = this.hook[hook].dependencies.map(x => x.replace('strapi-', '')) || [];;
+          let previousDependencies = this.hook[hook].dependencies.map(x => x.replace('strapi-', '')) || [];
 
           // Add BEFORE middlewares to load and remove the current one
           // to avoid that it waits itself.

scripts/setup.js

@@ -129,6 +129,7 @@ watcher('📦  Linking strapi-plugin-content-type-builder...', 'npm link --no-op
 watcher('🏗  Building...', 'npm run build');
 
 shell.cd('../strapi-plugin-graphql');
+watcher('', 'npm install ../strapi-utils');
 shell.rm('-f', 'package-lock.json');
 watcher('📦  Linking strapi-plugin-graphql...', 'npm link --no-optional', false);