Merge pull request #360 from roggervalf/set-husky

Adding husky, pretty-quick and lint-staged

.eslintrc.js

@@ -0,0 +1,46 @@
+module.exports = {
+  env: {
+    node: true,
+    es6: true,
+    browser: true,
+    commonjs: true,
+    jquery: true,
+  },
+  parserOptions: {
+    ecmaVersion: 2018,
+  },
+  plugins: ['prettier'],
+  extends: ['prettier'],
+  overrides: [
+    {
+      files: ['lib/**'],
+      rules: {
+        'max-len': 'error',
+      },
+    },
+    {
+      files: ['benchmark/**', 'examples/**'],
+      parserOptions: {
+        ecmaVersion: 8,
+      },
+      rules: {
+        'no-console': 'off',
+      },
+    },
+    {
+      files: ['test/**'],
+      parserOptions: {
+        ecmaVersion: 2017,
+        sourceType: 'module',
+      },
+      rules: {
+        'handle-callback-err': 'warn',
+        'no-shadow': 'off',
+        'no-warning-comments': 'off',
+      },
+    },
+  ],
+  rules: {
+    strict: 'off',
+  },
+};

.husky/.gitignore

@@ -0,0 +1 @@
+_

.husky/pre-commit

@@ -0,0 +1,5 @@
+#!/bin/sh
+. "$(dirname "$0")/_/husky.sh"
+
+npm run pretty:quick
+npm run lint:staged

.lintstagedrc

@@ -0,0 +1,3 @@
+{
+  "*.{js,md}": "node_modules/.bin/eslint . --ignore-path ./.prettierignore --fix"
+}

.prettierrc

@@ -1 +1,10 @@
-"@mixmaxhq/prettier-config"
+{
+  "tabWidth": 2,
+  "useTabs": false,
+  "semi": true,
+  "singleQuote": true,
+  "trailingComma": "es5",
+  "bracketSpacing": false,
+  "arrowParens": "always",
+  "endOfLine": "lf"
+}

example/bee.js

@@ -20,7 +20,7 @@ async function main() {
   });
 
   // Fake process function to move newly created jobs in the UI through a few of the job states.
-  queue.process(async function (job) {
+  queue.process(async function () {
     // Wait 5sec
     await new Promise((res) => setTimeout(res, 5000));
 

example/bull.js

@@ -21,7 +21,7 @@ async function main() {
   });
 
   // Fake process function to move newly created jobs in the UI through a few of the job states.
-  queue.process(async function (job) {
+  queue.process(async function () {
     // Wait 5sec
     await new Promise((res) => setTimeout(res, 5000));
 
@@ -32,7 +32,7 @@ async function main() {
   });
 
   // adding delayed jobs
-  const delayedJob = await queue.add({}, { delay: 60 * 1000 });
+  const delayedJob = await queue.add({}, {delay: 60 * 1000});
   delayedJob.log('Log message');
 
   const app = Arena(

example/bullmq.js

@@ -1,5 +1,5 @@
 const Arena = require('../');
-const { Queue, QueueScheduler, Worker } = require('bullmq');
+const {Queue, QueueScheduler, Worker} = require('bullmq');
 const RedisServer = require('redis-server');
 
 // Select ports that are unlikely to be used by other services a developer might be running locally.
@@ -14,17 +14,17 @@ async function main() {
   const queueName = 'name_of_my_queue';
 
   const queueScheduler = new QueueScheduler(queueName, {
-    connection: { port: REDIS_SERVER_PORT },
+    connection: {port: REDIS_SERVER_PORT},
   });
   await queueScheduler.waitUntilReady();
 
   const queue = new Queue(queueName, {
-    connection: { port: REDIS_SERVER_PORT },
+    connection: {port: REDIS_SERVER_PORT},
   });
 
   new Worker(
     queueName,
-    async function (job) {
+    async function () {
       // Wait 5sec
       await new Promise((res) => setTimeout(res, 5000));
 
@@ -34,12 +34,12 @@ async function main() {
       }
     },
     {
-      connection: { port: REDIS_SERVER_PORT },
+      connection: {port: REDIS_SERVER_PORT},
     }
   );
 
   // adding delayed jobs
-  const delayedJob = await queue.add('delayed', {}, { delay: 60 * 1000 });
+  const delayedJob = await queue.add('delayed', {}, {delay: 60 * 1000});
   delayedJob.log('Log message');
 
   Arena(

index.js

@@ -4,19 +4,25 @@ const Arena = require('./src/server/app');
 const routes = require('./src/server/views/routes');
 
 function run(config, listenOpts = {}) {
-  const { app, Queues } = Arena(config);
+  const {app, Queues} = Arena(config);
 
-  Queues.useCdn = typeof listenOpts.useCdn !== 'undefined' ? listenOpts.useCdn : true;
+  Queues.useCdn =
+    typeof listenOpts.useCdn !== 'undefined' ? listenOpts.useCdn : true;
 
   app.locals.appBasePath = listenOpts.basePath || app.locals.appBasePath;
 
-  app.use(app.locals.appBasePath, express.static(path.join(__dirname, 'public')));
+  app.use(
+    app.locals.appBasePath,
+    express.static(path.join(__dirname, 'public'))
+  );
   app.use(app.locals.appBasePath, routes);
 
   const port = listenOpts.port || 4567;
   const host = listenOpts.host || '0.0.0.0'; // Default: listen to all network interfaces.
   if (!listenOpts.disableListen) {
-    app.listen(port, host, () => console.log(`Arena is running on port ${port} at host ${host}`));
+    app.listen(port, host, () => {
+      console.log(`Arena is running on port ${port} at host ${host}`);
+    });
   }
 
   return app;