search

microsoft / node-pty

Fork pseudoterminals in Node.JS
Other
1.43k stars 232 forks source link

Published npm node module contains *.test.js #528

Closed aeschli closed 2 years ago

aeschli commented 2 years ago

The node-pty node module contains several .test.js files.

node-pty is part of the vscode build and I'm looking into reducing the number of files we ship wth.

|   +---node-pty
|   |   |   LICENSE
|   |   |   package.json
|   |   |   
|   |   +---build
|   |   |   \---Release
|   |   |           conpty.node
|   |   |           conpty_console_list.node
|   |   |           pty.node
|   |   |           winpty-agent.exe
|   |   |           winpty.dll
|   |   |           
|   |   \---lib
|   |       |   conpty_console_list_agent.js
|   |       |   eventEmitter2.js
|   |       |   eventEmitter2.test.js
|   |       |   index.js
|   |       |   interfaces.js
|   |       |   terminal.js
|   |       |   terminal.test.js
|   |       |   testUtils.test.js
|   |       |   types.js
|   |       |   unixTerminal.js
|   |       |   unixTerminal.test.js
|   |       |   utils.js
|   |       |   windowsConoutConnection.js
|   |       |   windowsPtyAgent.js
|   |       |   windowsPtyAgent.test.js
|   |       |   windowsTerminal.js
|   |       |   windowsTerminal.test.js
|   |       |   
|   |       +---shared
|   |       |       conout.js
|   |       |       
|   |       \---worker
|   |               conoutSocketWorker.js
Tyriar commented 2 years ago

Ignored in https://github.com/microsoft/node-pty/pull/530, it might be some time until we update node-pty though as there were problems integrating the latest version.