Add GH actions (#82)

* packages: bump node and add nvmrc

* packages: commit package-lock

* github-actions: add build / lint actions

* lint: ban console

* github-actions: fail build step on warnings

.eslintrc

@@ -24,6 +24,7 @@
       }
     ],
     "@typescript-eslint/no-explicit-any": ["error"],
+    "no-console": "error",
     "array-callback-return": ["error"],
     "padding-line-between-statements": [
       "warn",

.github/workflows/build.yml

@@ -0,0 +1,32 @@
+on:
+  pull_request:
+    types: [edited, synchronize, opened]
+  push:
+    branches: [master]
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: 'Use node version'
+        uses: actions/setup-node@v4
+        with:
+          node-version-file: .nvmrc
+          cache: 'npm'
+      - name: 'Install dependencies'
+        run: npm install
+      - name: 'Build'
+        run: npm run build-ci
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: 'Use node version'
+        uses: actions/setup-node@v4
+        with:
+          node-version-file: .nvmrc
+          cache: 'npm'
+      - name: 'Install dependencies'
+        run: npm install
+      - name: 'Linting with ESLint'
+        run: npm run lint

.gitignore

@@ -5,7 +5,6 @@ settings.json
 
 # npm
 node_modules
-package-lock.json
 
 # build
 build

.nvmrc

@@ -0,0 +1 @@
+v21.5.0

main.ts

@@ -25,7 +25,6 @@ const DEFAULT_SETTINGS: CustomJSSettings = {
   startupScriptNames: [],
   registeredInvocableScriptNames: [],
 };
-
 interface Invocable {
   invoke: () => Promise<void>;
 }
@@ -38,6 +37,7 @@ export default class CustomJS extends Plugin {
   settings: CustomJSSettings;
 
   async onload() {
+    // eslint-disable-next-line no-console
     console.log('Loading CustomJS');
     await this.loadSettings();
     this.registerEvent(this.app.vault.on('modify', this.reloadIfNeeded, this));
@@ -84,12 +84,14 @@ export default class CustomJS extends Plugin {
     const scriptObj = window.customJS[scriptName];
 
     if (!scriptObj) {
+      // eslint-disable-next-line no-console
       console.warn(`Script '${scriptName}' is not defined`);
 
       return;
     }
 
     if (!isInvocable(scriptObj)) {
+      // eslint-disable-next-line no-console
       console.warn(`Script '${scriptName}' is not invocable`);
 
       return;
@@ -103,7 +105,9 @@ export default class CustomJS extends Plugin {
       new Notice(
         `${message}\n${e.message}\nSee error console for more details`,
       );
+      // eslint-disable-next-line no-console
       console.error(message);
+      // eslint-disable-next-line no-console
       console.error(e);
     }
   }
@@ -144,7 +148,9 @@ export default class CustomJS extends Plugin {
       // Provide a way to create a new instance
       window.customJS[`create${def.name}Instance`] = () => new def();
     } catch (e) {
+      // eslint-disable-next-line no-console
       console.error(`CustomJS couldn't import ${f}`);
+      // eslint-disable-next-line no-console
       console.error(e);
     }
   }