Merge pull request #2106 from MartinFournier/fix/ci

Split GitHub workflow into multiple jobs to get clearer results

.github/workflows/ci.yml

@@ -1,4 +1,3 @@
-# This is a basic workflow to help you get started with Actions
 name: CI
 
 on:
@@ -13,39 +12,44 @@ on:
 
 jobs:
   build:
-    # The type of runner that the job will run on
+    name: Build
     runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [16.13.1]
-
     steps:
-      # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
-        - run: echo "The job was automatically triggered by a ${{ github.event_name }} event."
-        - run: echo "This job is now running on a ${{ runner.os }} server hosted by GitHub!"
-        - run: echo "The name of your branch is ${{ github.ref }} and your repository is ${{ github.repository }}."
       - uses: actions/checkout@v2
-        - run: echo "The ${{ github.repository }} repository has been cloned to the runner."
+      - name: Use Node.js 16.13.1
-        - run: echo "The workflow is now ready to test your code on the runner."
-        - name: Use Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v2
         with:
-            node-version: ${{ matrix.node-version }}
+          node-version: 16.13.1
+          cache: 'npm'
+      - name: Install npm dependencies
+        run: npm ci
+      - name: Build the production app
+        run: npm run build
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Use Node.js 16.13.1
+        uses: actions/setup-node@v2
+        with:
+          node-version: 16.13.1
+          cache: 'npm'
       - name: Install npm dependencies
         run: npm ci
-        - name: Run unit tests
-          run: npm run test
       - name: Run linter
         run: npm run lint:report
-        - name: Build the web app
+  test:
-          run: npm run build
+    name: Test
-
+    runs-on: ubuntu-latest
-        # Cannot build the electron app by default using ubuntu
+    steps:
-        # Wrapper command 'wine64' not found on the system. Consult your Linux distribution's package manager to determine how to install Wine.
+      - uses: actions/checkout@v2
-        # - name: Build the electron app
+      - name: Use Node.js 16.13.1
-        #   run: ./package.sh
+        uses: actions/setup-node@v2
-
+        with:
-        # Unable to properly run the cypress tests for now, they are throwing errors.
+          node-version: 16.13.1
-        # - name: Run the integration tests
+          cache: 'npm'
-        #   run: npm run cy:test
+      - name: Install npm dependencies
+        run: npm ci
+      - name: Run linter
+        run: npm run test

src/NetscriptFunctions/Singularity.ts

@@ -819,7 +819,7 @@ export function NetscriptSingularity(
 
       return player.getUpgradeHomeRamCost();
     },
-    workForCompany: function (companyName: any, focus: boolean = true): any {
+    workForCompany: function (companyName: any, focus = true): any {
       helper.updateDynamicRam("workForCompany", getRamCost("workForCompany"));
       helper.checkSingularityAccess("workForCompany", 2);
 
@@ -991,7 +991,7 @@ export function NetscriptSingularity(
       workerScript.log("joinFaction", () => `Joined the '${name}' faction.`);
       return true;
     },
-    workForFaction: function (name: any, type: any, focus: boolean = true): any {
+    workForFaction: function (name: any, type: any, focus = true): any {
       helper.updateDynamicRam("workForFaction", getRamCost("workForFaction"));
       helper.checkSingularityAccess("workForFaction", 2);
       getFaction("workForFaction", name);

src/Terminal/commands/common/editor.ts

@@ -24,7 +24,7 @@ export async function main(ns) {
 
 export function commonEditor(
   command: string,
-  { terminal, router, player, server, args }: EditorParameters,
+  { terminal, router, player, args }: EditorParameters,
   scriptEditorRouteOptions?: ScriptEditorRouteOptions,
 ): void {
   if (args.length < 1) {