Skip to content
This repository was archived by the owner on Jul 8, 2025. It is now read-only.

Commit d7c11a3

Browse files
authored
fix: build and bundle scripts (#180)
1 parent 426b8a2 commit d7c11a3

File tree

5 files changed

+16
-29
lines changed

5 files changed

+16
-29
lines changed

.github/workflows/ci-build.yml

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,9 @@ jobs:
1919
with:
2020
node-version: ${{ matrix.node-version }}
2121
- name: Install Dependencies
22-
run: npm install
23-
- name: Build
24-
run: npm run build
22+
run: npm ci
2523
- name: Invoke LSP entrypoint(server.js)
26-
run: node output/server.js
24+
run: node dist/server.js
2725
- name: Run the unit tests
2826
run: npm test -- --coverage
2927
- name: Upload coverage to Codecov
@@ -41,12 +39,9 @@ jobs:
4139
with:
4240
node-version: 12.x
4341
- name: Install Dependencies
44-
run: npm install
45-
- name: Build
46-
run: npm run build
42+
run: npm ci
4743
- name: Create local package tarball
4844
run: npm pack
49-
working-directory: output
5045
- name: pull vscode-extension code
5146
uses: actions/checkout@v2
5247
with:
@@ -58,7 +53,10 @@ jobs:
5853
run: npm ci
5954
working-directory: ./vscode-extension
6055
- name: Install local lsp
61-
run: npm i ../output/fabric8-analytics-lsp-server-*.tgz
56+
run: npm i ../fabric8-analytics-lsp-server-*.tgz
57+
working-directory: ./vscode-extension
58+
- name: Copy fabric8-analytics-lsp-server files to one level up(FIXME)
59+
run: cp -a node_modules/fabric8-analytics-lsp-server/dist/* node_modules/fabric8-analytics-lsp-server/
6260
working-directory: ./vscode-extension
6361
- name: Remove test files
6462
run: rm -fr test

.gitignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jspm_packages
3838

3939
# Others
4040
.idea/
41-
output/
41+
dist
4242

4343
# generated tarball and publishing scripts
4444
target/

package.json

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,6 @@
1818
}
1919
],
2020
"license": "Apache-2.0",
21-
"engines": {
22-
"node": "^12.0"
23-
},
2421
"keywords": [
2522
"fabric8-analytics",
2623
"LSP"
@@ -29,7 +26,11 @@
2926
"type": "git",
3027
"url": "https://github.com/fabric8-analytics/fabric8-analytics-lsp-server.git"
3128
},
32-
"main": "output/server.js",
29+
"main": "dist/server.js",
30+
"bin": "dist/server.js",
31+
"files": [
32+
"dist"
33+
],
3334
"dependencies": {
3435
"@xml-tools/ast": "^5.0.3",
3536
"@xml-tools/parser": "1.0.9",
@@ -57,9 +58,9 @@
5758
"typescript": "^3.6.3"
5859
},
5960
"scripts": {
60-
"build": "tsc -p . && cp package.json LICENSE README.md output",
61+
"build": "tsc -p .",
6162
"test": "nyc mocha",
62-
"semantic-release": "semantic-release --debug"
63+
"prepare": "npm run build"
6364
},
6465
"nyc": {
6566
"include": [

release.config.js

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
module.exports = {
22
prepare: [
3-
{
4-
path: "@semantic-release/exec",
5-
cmd: "npm run build && cp package-lock.json output",
6-
},
73
"@semantic-release/npm",
84
{
95
// build the Linux executable
@@ -27,17 +23,9 @@ module.exports = {
2723
},
2824
],
2925
publish: [
30-
{
31-
// install deps and create release tarball for intellij use.
32-
path: "@semantic-release/exec",
33-
cmd: "cd output && npm i --only=prod && echo ${nextRelease.version}>VERSION && tar cvjf ../ca-lsp-server.tar ."
34-
},
3526
{
3627
path: "@semantic-release/github",
3728
assets: [
38-
{
39-
path: "ca-lsp-server.tar"
40-
},
4129
{
4230
path: "analytics-lsp-linux",
4331
name: "analytics-lsp-linux",

tsconfig.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"module": "commonjs",
55
"moduleResolution": "node",
66
"sourceMap": true,
7-
"outDir": "output",
7+
"outDir": "dist",
88
"esModuleInterop": true,
99
"lib": [
1010
"ES2019.array",

0 commit comments

Comments
 (0)