Skip to content

Commit 990ff6c

Browse files
committed
dev: cleanup build scripts
Compile the node component to a single file, bundled with all the node modules inline. This fixes a vsce warning by dramatically reducing the number of files in the final .vsix asset.
1 parent 5cac3b8 commit 990ff6c

14 files changed

Lines changed: 180 additions & 399 deletions

.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
out
1+
dist
22
node_modules
33
*.vsix
44
sqls_bin
55
sqls
6+
.direnv

.vscode/launch.json

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,8 @@
1010
"type": "pwa-extensionHost",
1111
"request": "launch",
1212
"args": ["--extensionDevelopmentPath=${workspaceFolder}"],
13-
"outFiles": ["${workspaceFolder}/out/**/*.js"],
14-
"debugWebviews": {
15-
"urlFilter": "*notebookRenderer*"
16-
},
13+
"outFiles": ["${workspaceFolder}/dist/**/*.js"],
14+
"debugWebviews": true,
1715
"trace": true,
1816
"preLaunchTask": "npm: watch"
1917
},
@@ -22,7 +20,7 @@
2220
"type": "extensionHost",
2321
"request": "launch",
2422
"args": ["--extensionDevelopmentPath=${workspaceFolder}"],
25-
"outFiles": ["${workspaceFolder}/out/**/*.js"],
23+
"outFiles": ["${workspaceFolder}/dist/**/*.js"],
2624
"debugWebviews": true,
2725
"preLaunchTask": "npm: watch"
2826
}

.vscode/settings.json

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
{
22
"files.exclude": {
3-
"out": false // set this to true to hide the "out" folder with the compiled JS files
3+
"dist": false // set this to true to hide the "out" folder with the compiled JS files
44
},
55
"search.exclude": {
6-
"out": true // set this to false to include "out" folder in search results
6+
"dist": true
77
},
88
// Turn off tsc task auto detection since we have the necessary tasks as npm scripts
9-
"typescript.tsc.autoDetect": "off"
9+
"typescript.tsc.autoDetect": "off",
10+
"cSpell.enabled": true
1011
}

.vscode/tasks.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
{
77
"type": "npm",
88
"script": "watch",
9-
"problemMatcher": ["$tsc-watch", "$ts-checker-webpack-watch"],
109
"isBackground": true,
1110
"presentation": {
1211
"reveal": "never"
@@ -19,7 +18,6 @@
1918
{
2019
"type": "npm",
2120
"script": "compile",
22-
"problemMatcher": ["$tsc", "$ts-checker-webpack"],
2321
"group": "build"
2422
}
2523
]

.vscodeignore

Lines changed: 10 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,10 @@
1-
.vscode/**
2-
.vscode-test/**
3-
out/test/**
4-
src/**
5-
.gitignore
6-
**/tsconfig.json
7-
**/.eslintrc.json
8-
**/*.map
9-
**/*.ts
10-
**/*.tsbuildinfo
11-
sqls
12-
.github
13-
flake.nix
14-
flake.lock
15-
.envrc
16-
.gitmodules
17-
webview
1+
*
2+
**/**
3+
!LICENSE
4+
!CHANGELOG.md
5+
!dist/**/*
6+
!media/**/*
7+
!sqls_bin/**/*
8+
!README.md
9+
!package.json
10+
!package-lock.json

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
## v0.5.2
44

5-
- When clicking `Run All`, cells now execute in series. Previously, cells executed in parallel.
5+
- When clicking `Run All`, cells now execute in series. Previously, cells executed in parallel.
66

77
- New configuration option for maximum number of result rows before truncating the result table.
88
Defaults to `25`.

0 commit comments

Comments
 (0)