Configure eslint and lint all files

Author: mhaidarhanif

.eslintrc.js

@@ -1,29 +1,17 @@
 module.exports = {
-    "plugins": ['prettier'],
-    "env": {
-        "es6": true,
-        "node": true
-    },
-    "extends": "eslint:recommended",
-    "parserOptions": {
-        "sourceType": "module"
-    },
-    "rules": {
-        "indent": [
-            "error",
-            2
-        ],
-        "linebreak-style": [
-            "error",
-            "unix"
-        ],
-        "quotes": [
-            "error",
-            "single"
-        ],
-        "semi": [
-            "error",
-            "never"
-        ]
-    }
-};
+  plugins: ['prettier'],
+  env: {
+    es6: true,
+    node: true
+  },
+  parserOptions: {
+    sourceType: 'module',
+    ecmaVersion: 2017
+  },
+  rules: {
+    indent: ['error', 2],
+    'linebreak-style': ['error', 'unix'],
+    quotes: ['error', 'single'],
+    semi: ['error', 'never']
+  }
+}

api/index.js

@@ -1,11 +1,11 @@
-var express = require('express');
-var router = express.Router();
+var express = require('express')
+var router = express.Router()
 
 /* GET home page. */
-router.get('/', function (req, res, next) {
+router.get('/', function(req, res, next) {
   res.send({
-    title: 'Express'
-  });
-});
+    message: `Welcome to REST API with ORM and SQL Database`
+  })
+})
 
-module.exports = router;
+module.exports = router

app.js

@@ -1,40 +1,44 @@
-const createError = require('http-errors');
-const express = require('express');
-const path = require('path');
-const cookieParser = require('cookie-parser');
-const logger = require('morgan');
-
-const indexRouter = require('./api/index');
-const usersRouter = require('./api/users');
-
-const app = express();
-
-app.use(logger('dev'));
-app.use(express.json());
-app.use(express.urlencoded({
-  extended: false
-}));
-app.use(cookieParser());
-
-app.use('/', indexRouter);
-app.use('/users', usersRouter);
+const createError = require('http-errors')
+const express = require('express')
+const path = require('path')
+const cookieParser = require('cookie-parser')
+const logger = require('morgan')
+const cors = require('cors')
+
+const indexRouter = require('./api/index')
+const usersRouter = require('./api/users')
+
+const app = express()
+
+app.use(cors())
+app.use(logger('dev'))
+app.use(express.json())
+app.use(
+  express.urlencoded({
+    extended: false
+  })
+)
+app.use(cookieParser())
+
+app.use('/', indexRouter)
+app.use('/users', usersRouter)
 
 // catch 404 and forward to error handler
-app.use(function (req, res, next) {
-  next(createError(404));
-});
+app.use(function(req, res, next) {
+  next(createError(404))
+})
 
 // error handler
-app.use(function (err, req, res, next) {
+app.use(function(err, req, res, next) {
   // set locals, only providing error in development
-  res.locals.message = err.message;
-  res.locals.error = req.app.get('env') === 'development' ? err : {};
+  res.locals.message = err.message
+  res.locals.error = req.app.get('env') === 'development' ? err : {}
 
   // render the error page
-  res.status(err.status || 500);
+  res.status(err.status || 500)
   res.send({
     err
-  });
-});
+  })
+})
 
-module.exports = app;
+module.exports = app

index.js

@@ -10,64 +10,64 @@ require('dotenv-extended').load({
   errorOnExtra: false,
   assignToProcessEnv: true,
   overrideProcessEnv: false
-});
+})
 
 /**
  * Module dependencies.
  */
 
-const app = require('./app');
-const debug = require('debug')('code-express-orm-mariadb:server');
-const http = require('http');
-const models = require('./models');
+const app = require('./app')
+const debug = require('debug')('code-express-orm-mariadb:server')
+const http = require('http')
+const models = require('./models')
 
 /**
  * Get port from environment and store in Express.
  */
 
-const port = normalizePort(process.env.PORT || '3000');
-app.set('port', port);
+const port = normalizePort(process.env.PORT || '3000')
+app.set('port', port)
 
 /**
  * Create HTTP server.
  */
 
-const server = http.createServer(app);
+const server = http.createServer(app)
 
 /**
  * Listen on provided port, on all network interfaces.
  */
 
-models.sequelize.sync().then(function () {
+models.sequelize.sync().then(function() {
   /**
    * Listen on provided port, on all network interfaces.
    */
-  server.listen(port, function () {
-    console.log('Express server listening on port ' + server.address().port);
-    debug('Express server listening on port ' + server.address().port);
-  });
-  server.on('error', onError);
-  server.on('listening', onListening);
-});
+  server.listen(port, function() {
+    console.log('Express server listening on port ' + server.address().port)
+    debug('Express server listening on port ' + server.address().port)
+  })
+  server.on('error', onError)
+  server.on('listening', onListening)
+})
 
 /**
  * Normalize a port into a number, string, or false.
  */
 
 function normalizePort(val) {
-  const port = parseInt(val, 10);
+  const port = parseInt(val, 10)
 
   if (isNaN(port)) {
     // named pipe
-    return val;
+    return val
   }
 
   if (port >= 0) {
     // port number
-    return port;
+    return port
   }
 
-  return false;
+  return false
 }
 
 /**
@@ -76,25 +76,23 @@ function normalizePort(val) {
 
 function onError(error) {
   if (error.syscall !== 'listen') {
-    throw error;
+    throw error
   }
 
-  const bind = typeof port === 'string' ?
-    'Pipe ' + port :
-    'Port ' + port;
+  const bind = typeof port === 'string' ? 'Pipe ' + port : 'Port ' + port
 
   // handle specific listen errors with friendly messages
   switch (error.code) {
-    case 'EACCES':
-      console.error(bind + ' requires elevated privileges');
-      process.exit(1);
-      break;
-    case 'EADDRINUSE':
-      console.error(bind + ' is already in use');
-      process.exit(1);
-      break;
-    default:
-      throw error;
+  case 'EACCES':
+    console.error(bind + ' requires elevated privileges')
+    process.exit(1)
+    break
+  case 'EADDRINUSE':
+    console.error(bind + ' is already in use')
+    process.exit(1)
+    break
+  default:
+    throw error
   }
 }
 
@@ -103,9 +101,7 @@ function onError(error) {
  */
 
 function onListening() {
-  const addr = server.address();
-  const bind = typeof addr === 'string' ?
-    'pipe ' + addr :
-    'port ' + addr.port;
-  debug('Listening on ' + bind);
-}
+  const addr = server.address()
+  const bind = typeof addr === 'string' ? 'pipe ' + addr : 'port ' + addr.port
+  debug('Listening on ' + bind)
+}

migrations/1-create-user.js

@@ -1,4 +1,5 @@
-'use strict';
+'use strict'
+
 module.exports = {
   up: (queryInterface, Sequelize) => {
     return queryInterface.createTable('Users', {
@@ -26,9 +27,9 @@ module.exports = {
         type: Sequelize.DATE,
         allowNull: false
       }
-    });
+    })
   },
-  down: (queryInterface) => {
-    return queryInterface.dropTable('Users');
+  down: queryInterface => {
+    return queryInterface.dropTable('Users')
   }
-};
+}

package.json

@@ -7,24 +7,32 @@
     "migrate": "sequelize db:migrate && sequelize db:seed:all",
     "migrate:undo": "sequelize db:seed:undo:all && sequelize db:migrate:undo:all",
     "start": "node index",
-    "dev": "node-dev index"
+    "dev": "node-dev index",
+    "test": "node test",
+    "lint": "eslint *.js",
+    "precommit": "pretty-quick --staged"
   },
   "dependencies": {
+    "body-parser": "^1.18.3",
     "cookie-parser": "~1.4.3",
+    "cors": "^2.8.4",
     "debug": "~2.6.9",
     "dotenv-extended": "^2.2.0",
     "express": "~4.16.0",
     "http-errors": "~1.6.2",
     "jade": "~1.11.0",
     "morgan": "~1.9.0",
     "mysql2": "^1.6.1",
+    "prettier": "^1.14.2",
+    "pretty-quick": "^1.6.0",
     "sequelize": "^4.38.0"
   },
   "devDependencies": {
     "eslint": "^5.3.0",
     "eslint-config-prettier": "^2.9.0",
     "eslint-plugin-prettier": "^2.6.2",
     "node-dev": "^3.1.3",
+    "prettier-eslint": "^8.8.2",
     "sequelize-cli": "^4.0.0"
   }
 }