[feat][ci] implement linting using GitHub actions

.eslintrc.js

@@ -0,0 +1,42 @@
+module.exports = {
+  env: {
+    es2021: true,
+    node: true,
+  },
+  extends: [
+    "eslint:recommended",
+    "plugin:react/recommended",
+  ],
+  settings: {
+    react: {
+      version: "18.2",
+    },
+  },
+  overrides: [
+  ],
+  parserOptions: {
+    ecmaVersion: "latest",
+    sourceType: "module",
+  },
+  plugins: [
+    "react",
+  ],
+  rules: {
+    indent: [
+      "error",
+      2,
+    ],
+    "linebreak-style": [
+      "error",
+      "unix",
+    ],
+    quotes: [
+      "error",
+      "double",
+    ],
+    semi: [
+      "error",
+      "always",
+    ],
+  },
+};

.github/workflows/lint.yml

@@ -0,0 +1,22 @@
+name: lint
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - name: Setup Node and cache dependencies
+      uses: actions/setup-node@v3
+      with:
+        node-version: 16
+        cache: 'npm'
+    - name: install dependencies
+      run: npm install
+    - name: Lint files
+      run: npm run lint

.gitignore

@@ -1,3 +1,4 @@
-*/node_modules
+node_modules
 */.env
 *.sqlite
+.vscode

api-gateway/index.js

@@ -1,35 +1,35 @@
-import express from 'express';
-import cors from 'cors';
-import morgan from 'morgan';
-import cookieParser from 'cookie-parser';
-import 'dotenv/config';
+import express from "express";
+import cors from "cors";
+import morgan from "morgan";
+import cookieParser from "cookie-parser";
+import "dotenv/config";
 import {
   userProxy,
   matchingProxy,
   codingProxy,
   videoProxy,
   questionProxy,
   historyProxy,
-} from './proxy.js';
+} from "./proxy.js";
 
 const app = express();
 app.use(
   cors({
-    //replace with deployed endpoint
-    origin: 'http://localhost:3000',
+    // replace with deployed endpoint
+    origin: "http://localhost:3000",
     credentials: true,
-  })
+  }),
 ); // config cors so that front-end can use
 app.use(cookieParser());
-app.use(morgan('combined'));
+app.use(morgan("combined"));
 
-app.use('/user', userProxy);
+app.use("/user", userProxy);
 app.use(matchingProxy);
 app.use(codingProxy);
-app.use('/video', videoProxy);
-app.use('/question', questionProxy);
-app.use('/history', historyProxy);
+app.use("/video", videoProxy);
+app.use("/question", questionProxy);
+app.use("/history", historyProxy);
 
 const server = app.listen(8080, () => {
-  console.log('api gateway running on port 8080');
+  console.log("api gateway running on port 8080");
 });

api-gateway/middleware/auth.js

@@ -1,20 +1,20 @@
-import axios from 'axios';
+import axios from "axios";
 
 export const authenticate = async (req, res, next) => {
   const { token } = req.cookies;
   await axios
-    .get('http://localhost:8000/auth', {
+    .get("http://localhost:8000/auth", {
       headers: {
         Cookie: `token=${token}`,
       },
     })
     .then((resp) => {
       if (resp.status != 200) {
-        res.status(401).json({ message: 'Invalid JWT token' });
+        res.status(401).json({ message: "Invalid JWT token" });
       }
       next();
     })
-    .catch(() => res.status(401).json({ message: 'Missing JWT token' }));
+    .catch(() => res.status(401).json({ message: "Missing JWT token" }));
 };
 
 export default authenticate;

api-gateway/proxy.js

@@ -1,17 +1,17 @@
-import { createProxyMiddleware } from 'http-proxy-middleware';
+import { createProxyMiddleware } from "http-proxy-middleware";
 
 export const userProxy = createProxyMiddleware({
   target: process.env.USER_URL,
   changeOrigin: true,
 });
 
-export const matchingProxy = createProxyMiddleware('/matching', {
+export const matchingProxy = createProxyMiddleware("/matching", {
   target: process.env.MATCHING_URL,
   changeOrigin: true,
   ws: true,
 });
 
-export const codingProxy = createProxyMiddleware('/coding', {
+export const codingProxy = createProxyMiddleware("/coding", {
   target: process.env.CODING_URL,
   changeOrigin: true,
   ws: true,

coding-service/index.js

@@ -1,40 +1,40 @@
-import express from 'express';
-import cors from 'cors';
-import http from 'http';
-import { Server } from 'socket.io';
+import express from "express";
+import cors from "cors";
+import http from "http";
+import { Server } from "socket.io";
 
 const app = express();
 app.use(express.urlencoded({ extended: true }));
 app.use(express.json());
 app.use(cors()); // config cors so that front-end can use
-app.options('*', cors());
+app.options("*", cors());
 
 const server = http.createServer(app);
 const io = new Server(server, {
   cors: {
-    origin: '*',
+    origin: "*",
   },
-  path: '/coding',
+  path: "/coding",
 });
 
-var PORT = process.env.PORT || 8002;
+const PORT = process.env.PORT || 8002;
 
-app.get('/coding', (req, res) => {
-  res.send('Hello World from coding-service');
+app.get("/coding", (req, res) => {
+  res.send("Hello World from coding-service");
 });
 
-io.on('connection', (socket) => {
-  console.log('a user connected to coding-service');
+io.on("connection", (socket) => {
+  console.log("a user connected to coding-service");
 
-  socket.on('connectedToRoom', (roomId) => {
+  socket.on("connectedToRoom", (roomId) => {
     const roomName = `ROOM:${roomId}`;
     socket.join(roomName);
   });
 
-  socket.on('codeChanged', (args) => {
+  socket.on("codeChanged", (args) => {
     const { value, roomId } = args;
     const roomName = `ROOM:${roomId}`;
-    socket.in(roomName).emit('codeChanged', value);
+    socket.in(roomName).emit("codeChanged", value);
   });
 });
 

frontend/package.json

@@ -35,12 +35,6 @@
     "test": "react-scripts test",
     "lint": "npx eslint ."
   },
-  "eslintConfig": {
-    "extends": [
-      "react-app",
-      "react-app/jest"
-    ]
-  },
   "browserslist": {
     "production": [
       ">0.2%",