add eslint

.eslintrc

@@ -0,0 +1,41 @@
+{
+  "parser": "@typescript-eslint/parser",
+  "plugins": ["import", "unused-imports"],
+  "extends": [
+    "eslint:recommended",
+    "plugin:@typescript-eslint/recommended",
+    "plugin:prettier/recommended"
+  ],
+  "rules": {
+    "@typescript-eslint/no-explicit-any": "off",
+    "unused-imports/no-unused-imports": "error",
+    "@typescript-eslint/no-unused-vars": [
+      "error", { "argsIgnorePattern": "^_" }],
+    "prettier/prettier": [
+      "error",
+      {
+        "arrowParens": "avoid",
+        "singleQuote": true,
+        "semi": true,
+        "tabWidth": 2,
+        "useTabs": false,
+        "trailingComma": "all",
+        "jsdoc-format": false,
+        "endOfLine": "auto"
+      }
+    ],
+    "import/order": [
+      "error",
+      {
+        "groups": [
+          "builtin",
+          "external",
+          ["internal", "parent", "sibling"],
+          "index",
+          "object",
+          "type"
+        ]
+      }
+    ]
+  }
+}

.github/workflows/main-pipeline.yml

@@ -43,7 +43,7 @@ jobs:
       - name: Use Node.js
         uses: actions/setup-node@v1
         with:
-          node-version: 18.14.0
+          node-version: 20.11.0
       - name: Install dependencies
         run: npm ci
       - name: Run linter

.github/workflows/staging-pipeline.yml

@@ -41,15 +41,22 @@ jobs:
       - name: Use Node.js
         uses: actions/setup-node@v1
         with:
-          node-version: 18.14.0
+          node-version: 20.14.0
+
+      - name: Install Yarn
+        run: npm install -g yarn
+
       - name: Install dependencies
-        run: npm ci
+        run: yarn install
+
       - name: Run linter
-        run: npm run lint
+        run: yarn lint
+
       - name: Run migrations
-        run: npm run db:migrate:run:test
+        run: yarn db:migrate:run:test
+
       - name: Run tests
-        run: npm run test
+        run: yarn test
 
   publish:
     needs: test
@@ -58,6 +65,13 @@ jobs:
     steps:
       - name: Check out the repo
         uses: actions/checkout@v2
+
+      - name: Install Yarn
+        run: npm install -g yarn
+
+      - name: Install dependencies
+        run: yarn install
+
       - name: Build image and push to GitHub Packages
         uses: docker/build-push-action@v1
         with:

migrations/1678175596807-changeNotificationCopies.ts

@@ -24,7 +24,7 @@ export class changeNotificationCopies1678175596807
     await notificationType.save();
   }
 
-  public async up(queryRunner: QueryRunner): Promise<void> {
+  public async up(_queryRunner: QueryRunner): Promise<void> {
     if (
       process.env.NODE_ENV === 'test' ||
       process.env.NODE_ENV === 'development'
@@ -339,7 +339,7 @@ export class changeNotificationCopies1678175596807
     );
   }
 
-  public async down(queryRunner: QueryRunner): Promise<void> {
+  public async down(_queryRunner: QueryRunner): Promise<void> {
     //
   }
 }

migrations/1681206838965-changeCancelProjectNotificationCopies.ts

@@ -21,7 +21,7 @@ export class changeCancelProjectNotificationCopies1681206838965
     await notificationType.save();
   }
 
-  public async up(queryRunner: QueryRunner): Promise<void> {
+  public async up(_queryRunner: QueryRunner): Promise<void> {
     if (
       process.env.NODE_ENV === 'test' ||
       process.env.NODE_ENV === 'development'
@@ -84,7 +84,7 @@ export class changeCancelProjectNotificationCopies1681206838965
     );
   }
 
-  public async down(queryRunner: QueryRunner): Promise<void> {
+  public async down(_queryRunner: QueryRunner): Promise<void> {
     //
   }
 }

migrations/1682242459196-seedNotificationTypeForProjectGetNewRank.ts

@@ -3,13 +3,12 @@ import {
   NotificationType,
   SCHEMA_VALIDATORS_NAMES,
 } from '../src/entities/notificationType';
-import { MICRO_SERVICES, THIRD_PARTY_EMAIL_SERVICES } from '../src/utils/utils';
+import { MICRO_SERVICES } from '../src/utils/utils';
 import {
   NOTIFICATION_CATEGORY,
   NOTIFICATION_TYPE_NAMES,
 } from '../src/types/general';
 import { NOTIFICATION_CATEGORY_GROUPS } from '../src/entities/notificationSetting';
-import { SegmentEvents } from '../src/services/segment/segmentAnalyticsSingleton';
 
 // https://github.com/Giveth/notification-center/issues/6 , https://gist.github.com/MohammadPCh/24434d50bc9ccd9b74905c271ee05482
 // icons https://gist.github.com/MohammadPCh/31e2b750dd9aa54edb21dcc6e7332efb

migrations/1692698983844-changeNotificationCopies.ts

@@ -21,7 +21,7 @@ export class changeNotificationCopies1692698983844
     await notificationType.save();
   }
 
-  public async up(queryRunner: QueryRunner): Promise<void> {
+  public async up(_queryRunner: QueryRunner): Promise<void> {
     if (
       process.env.NODE_ENV === 'test' ||
       process.env.NODE_ENV === 'development'
@@ -85,7 +85,7 @@ export class changeNotificationCopies1692698983844
     );
   }
 
-  public async down(queryRunner: QueryRunner): Promise<void> {
+  public async down(_queryRunner: QueryRunner): Promise<void> {
     //
   }
 }

migrations/1712683625687-addSuperFluidNotificationForAllUsers.ts

@@ -1,21 +1,23 @@
-import { MigrationInterface, QueryRunner } from "typeorm"
+import { MigrationInterface, QueryRunner } from 'typeorm';
 
-export class addSuperFluidNotificationForAllUsers1712683625687 implements MigrationInterface {
-    public async up(queryRunner: QueryRunner): Promise<void> {
-        // Fetch the notificationTypeIds for the "superfluid" categoryGroup
-        const notificationTypeIds = await queryRunner.query(`
+export class addSuperFluidNotificationForAllUsers1712683625687
+  implements MigrationInterface
+{
+  public async up(queryRunner: QueryRunner): Promise<void> {
+    // Fetch the notificationTypeIds for the "superfluid" categoryGroup
+    const notificationTypeIds = await queryRunner.query(`
             SELECT "id" FROM "notification_type" WHERE "categoryGroup" = 'superfluid';
         `);
 
-        // Fetch all unique userAddressIds
-        const userAddressIds = await queryRunner.query(`
+    // Fetch all unique userAddressIds
+    const userAddressIds = await queryRunner.query(`
             SELECT DISTINCT "userAddressId" FROM "notification_setting";
         `);
 
-        // For each userAddressId, insert a new row for each notificationTypeId
-        for (const { userAddressId } of userAddressIds) {
-            for (const { id: notificationTypeId } of notificationTypeIds) {
-                await queryRunner.query(`
+    // For each userAddressId, insert a new row for each notificationTypeId
+    for (const { userAddressId } of userAddressIds) {
+      for (const { id: notificationTypeId } of notificationTypeIds) {
+        await queryRunner.query(`
                     INSERT INTO "notification_setting" (
                         "allowNotifications", 
                         "allowEmailNotification", 
@@ -24,22 +26,22 @@ export class addSuperFluidNotificationForAllUsers1712683625687 implements Migrat
                         "userAddressId"
                     ) VALUES (true, true, true, ${notificationTypeId}, ${userAddressId});
                 `);
-            }
-        }
+      }
     }
+  }
 
-    public async down(queryRunner: QueryRunner): Promise<void> {
-        // Fetch the notificationTypeIds for the "superfluid" categoryGroup
-        const notificationTypeIds = await queryRunner.query(`
+  public async down(queryRunner: QueryRunner): Promise<void> {
+    // Fetch the notificationTypeIds for the "superfluid" categoryGroup
+    const notificationTypeIds = await queryRunner.query(`
             SELECT "id" FROM "notification_type" WHERE "categoryGroup" = 'superfluid';
         `);
 
-        // Convert fetched rows to a list of IDs for the IN clause
-        const ids = notificationTypeIds.map((nt: { id: any; }) => nt.id).join(', ');
+    // Convert fetched rows to a list of IDs for the IN clause
+    const ids = notificationTypeIds.map((nt: { id: any }) => nt.id).join(', ');
 
-        // Delete the rows with the fetched notificationTypeIds for all userAddressIds
-        await queryRunner.query(`
+    // Delete the rows with the fetched notificationTypeIds for all userAddressIds
+    await queryRunner.query(`
             DELETE FROM "notification_setting" WHERE "notificationTypeId" IN (${ids});
         `);
-    }
+  }
 }

migrations/1713834025817-addGroupParentSuperFluid.ts

@@ -1,51 +1,55 @@
-import { MigrationInterface, QueryRunner } from "typeorm"
+import { MigrationInterface, QueryRunner } from 'typeorm';
 import { NOTIFICATION_CATEGORY } from '../src/types/general';
 import { NotificationType } from '../src/entities/notificationType';
 import { MICRO_SERVICES } from '../src/utils/utils';
 import { NOTIFICATION_CATEGORY_GROUPS } from '../src/entities/notificationSetting';
 
 export const superFluidNotificationTypes = [
-    {
-        isGlobal: false,
-        isGroupParent: true,
-        showOnSettingPage: true,
-        webDefaultValue: true,
-        emailDefaultValue: true,
-        isEmailEditable: true,
-        isWebEditable: true,
-        name: 'Stream balance warnings',
-        description: 'Notify me when any of my Stream Balances are running low',
-        microService: MICRO_SERVICES.givethio,
-        category: NOTIFICATION_CATEGORY.SUPPORTED_PROJECTS,
-        schemaValidator: null,
-        emailNotifierService: null,
-        emailNotificationId: null,
-        pushNotifierService: null,
-        categoryGroup: NOTIFICATION_CATEGORY_GROUPS.SUPERFLUID,
-        title: 'Stream balance warnings',
-    },
-]
+  {
+    isGlobal: false,
+    isGroupParent: true,
+    showOnSettingPage: true,
+    webDefaultValue: true,
+    emailDefaultValue: true,
+    isEmailEditable: true,
+    isWebEditable: true,
+    name: 'Stream balance warnings',
+    description: 'Notify me when any of my Stream Balances are running low',
+    microService: MICRO_SERVICES.givethio,
+    category: NOTIFICATION_CATEGORY.SUPPORTED_PROJECTS,
+    schemaValidator: null,
+    emailNotifierService: null,
+    emailNotificationId: null,
+    pushNotifierService: null,
+    categoryGroup: NOTIFICATION_CATEGORY_GROUPS.SUPERFLUID,
+    title: 'Stream balance warnings',
+  },
+];
 
-export class addGroupParentSuperFluid1713834025817 implements MigrationInterface {
-    public async up(queryRunner: QueryRunner): Promise<void> {
-        // Fetch the notificationTypeIds for the "superfluid" categoryGroup
-        const notificationTypeIds = (await queryRunner.query(`
+export class addGroupParentSuperFluid1713834025817
+  implements MigrationInterface
+{
+  public async up(queryRunner: QueryRunner): Promise<void> {
+    // Fetch the notificationTypeIds for the "superfluid" categoryGroup
+    const notificationTypeIds = (
+      await queryRunner.query(`
             SELECT "id" FROM "notification_type" WHERE "categoryGroup" = 'superfluid';
-        `)).map((i: any) => i.id);
-        await queryRunner.query(
-          `UPDATE notification_type 
+        `)
+    ).map((i: any) => i.id);
+    await queryRunner.query(
+      `UPDATE notification_type 
              SET "showOnSettingPage" = false, "emailDefaultValue" = true, "category" = '${NOTIFICATION_CATEGORY.SUPPORTED_PROJECTS}'
-             WHERE id IN (${notificationTypeIds.join(", ")})`
-        );
-        await queryRunner.manager.save(
-          NotificationType,
-          superFluidNotificationTypes,
-        );
-    }
+             WHERE id IN (${notificationTypeIds.join(', ')})`,
+    );
+    await queryRunner.manager.save(
+      NotificationType,
+      superFluidNotificationTypes,
+    );
+  }
 
-    public async down(queryRunner: QueryRunner): Promise<void> {
-        await queryRunner.query(
-          `DELETE FROM notification_type WHERE "categoryGroup" = 'superfluid' AND "name" = 'Stream Balance Warnings';`,
-        );
-    }
+  public async down(queryRunner: QueryRunner): Promise<void> {
+    await queryRunner.query(
+      `DELETE FROM notification_type WHERE "categoryGroup" = 'superfluid' AND "name" = 'Stream Balance Warnings';`,
+    );
+  }
 }