Merge pull request #17 from mbehr1/fix/typo_and_pkg_upd

fix: typo vsc-lfs

.eslintrc.json

@@ -9,8 +9,7 @@
         "@typescript-eslint"
     ],
     "rules": {
-        "@typescript-eslint/class-name-casing": "warn",
-        "@typescript-eslint/semi": "warn",
+        "@typescript-eslint/semi": "off",
         "curly": "warn",
         "eqeqeq": "warn",
         "no-throw-literal": "warn",

.github/workflows/commitlint.yml

@@ -10,7 +10,7 @@ jobs:
   commitlint:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
         with:
           fetch-depth: 0
-      - uses: wagoid/commitlint-github-action@v2
+      - uses: wagoid/commitlint-github-action@v5

.github/workflows/ext_unit_tests.yml

@@ -0,0 +1,25 @@
+name: run ext unit tests
+on:
+  pull_request:
+    branches: [master]
+jobs:
+  build:
+    strategy:
+      matrix:
+        os: [macos-latest, ubuntu-latest]
+    runs-on: ${{ matrix.os }}
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v3
+      with:
+        fetch-depth: 0
+    - name: Install Node.js
+      uses: actions/setup-node@v3
+      with:
+        node-version: "lts/*"
+    - name: Install dependencies
+      run: npm install
+    - run: xvfb-run -a npm run test
+      if: runner.os == 'Linux'
+    - run: npm run test
+      if: runner.os != 'Linux'

.github/workflows/release.yml

@@ -8,15 +8,15 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0
       - name: Commitlint
-        uses: wagoid/commitlint-github-action@v2
-      - name: Setup Node.js
-        uses: actions/setup-node@v1
+        uses: wagoid/commitlint-github-action@v5
+      - name: Setup Node.js lts
+        uses: actions/setup-node@v3
         with:
-          node-version: 12
+          node-version: "lts/*"
       - name: Install dependencies
         run: npm ci
       - name: Release

.vscode/extensions.json

@@ -1,7 +1,8 @@
 {
-	// See http://go.microsoft.com/fwlink/?LinkId=827846
-	// for the documentation about the extensions.json format
-	"recommendations": [
-		"dbaeumer.vscode-eslint"
-	]
-}
+		// See http://go.microsoft.com/fwlink/?LinkId=827846
+		// for the documentation about the extensions.json format
+		"recommendations": [
+			"connor4312.esbuild-problem-matchers",
+			"dbaeumer.vscode-eslint"
+		]
+	}

.vscode/tasks.json

@@ -4,17 +4,42 @@
 	"version": "2.0.0",
 	"tasks": [
 		{
-			"type": "npm",
-			"script": "watch",
-			"problemMatcher": "$tsc-watch",
-			"isBackground": true,
+			"label": "watch",
+			"dependsOn": [
+				"npm: watch:tsc",
+				"npm: watch:esbuild"
+			],
 			"presentation": {
 				"reveal": "never"
 			},
 			"group": {
 				"kind": "build",
 				"isDefault": true
 			}
+		},
+		{
+			"type": "npm",
+			"script": "watch:esbuild",
+			"group": "build",
+			"problemMatcher": "$esbuild-watch",
+			"isBackground": true,
+			"label": "npm: watch:esbuild",
+			"presentation": {
+				"group": "watch",
+				"reveal": "never"
+			}
+		},
+		{
+			"type": "npm",
+			"script": "watch:tsc",
+			"group": "build",
+			"problemMatcher": "$tsc-watch",
+			"isBackground": true,
+			"label": "npm: watch:tsc",
+			"presentation": {
+				"group": "watch",
+				"reveal": "never"
+			}
 		}
 	]
 }

.vscodeignore

@@ -1,3 +1,5 @@
+node_modules/
+esbuild.mjs
 .vscode/**
 .vscode-test/**
 out/test/**

esbuild.mjs

@@ -0,0 +1,55 @@
+import * as esbuild from 'esbuild'
+
+const production = process.argv.includes('--production')
+const watch = process.argv.includes('--watch')
+
+async function main() {
+  const ctx = await esbuild.context({
+    entryPoints: [{ out: 'extension', in: 'src/extension.ts' }],
+    bundle: true,
+    format: 'cjs',
+    minify: production,
+    sourcemap: !production,
+    sourcesContent: false,
+    platform: 'node',
+    // outdir: 'out',
+    outfile: 'out/extension.js', // .cjs leads to errors in vsce package...
+    external: ['vscode'],
+    logLevel: 'silent',
+    plugins: [
+      /* add to the end of plugins array */
+      esbuildProblemMatcherPlugin,
+    ],
+  })
+  if (watch) {
+    await ctx.watch()
+  } else {
+    await ctx.rebuild()
+    await ctx.dispose()
+  }
+}
+
+/**
+ * @type {import('esbuild').Plugin}
+ */
+const esbuildProblemMatcherPlugin = {
+  name: 'esbuild-problem-matcher',
+
+  setup(build) {
+    build.onStart(() => {
+      console.log('[watch] build started')
+    })
+    build.onEnd((result) => {
+      result.errors.forEach(({ text, location }) => {
+        console.error(`✘ [ERROR] ${text}`)
+        console.error(`    ${location.file}:${location.line}:${location.column}:`)
+      })
+      console.log('[watch] build finished')
+    })
+  },
+}
+
+main().catch((e) => {
+  console.error(e)
+  process.exit(1)
+})