Skip to content

Commit 69b6401

Browse files
committed
chore: don't copy vtsls/typescript licenses in CI build
src-node deps aren't installed in the pipeline (the desktop app build installs them separately), so the copy would fail. The license files are already vendored under src/thirdparty/licences/; comment out the copy with a note.
1 parent 6f48130 commit 69b6401

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

gulpfile.js/thirdparty-lib-copy.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -265,11 +265,11 @@ let copyThirdPartyLibs = series(
265265
'addon-webgl.js', 'src/thirdparty/xterm'),
266266
renameFile.bind(renameFile, 'node_modules/@xterm/addon-webgl/lib/addon-webgl.js.map',
267267
'addon-webgl.js.map', 'src/thirdparty/xterm'),
268-
copyLicence.bind(copyLicence, 'node_modules/@xterm/xterm/LICENSE', 'xterm'),
269-
// vtsls language server (bundled in src-node for desktop LSP) + its bundled typescript
270-
copyLicence.bind(copyLicence, 'src-node/node_modules/@vtsls/language-server/LICENSE', 'vtsls'),
271-
copyLicence.bind(copyLicence, 'src-node/node_modules/typescript/LICENSE.txt', 'typescript')
272-
268+
copyLicence.bind(copyLicence, 'node_modules/@xterm/xterm/LICENSE', 'xterm')
269+
// vtsls language server (bundled in src-node is not installed in pipline tests as the destop app building
270+
// does it for desktop LSP). we ran it once and copied the license here.
271+
// copyLicence.bind(copyLicence, 'src-node/node_modules/@vtsls/language-server/LICENSE', 'vtsls'),
272+
// copyLicence.bind(copyLicence, 'src-node/node_modules/typescript/LICENSE.txt', 'typescript')
273273
);
274274

275275
/**

0 commit comments

Comments
 (0)