Skip to content

Update node to 10.15.1 #472

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
code-asher merged 11 commits into from
Apr 16, 2019
Merged

Update node to 10.15.1 #472

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
ce968b6
Update Node to 10.15.1
code-asher Apr 9, 2019
c42b63e
Remove string replace that was used for oclif
code-asher Apr 9, 2019
aa7956f
Update nbin
code-asher Apr 9, 2019
d0e497f
Package node-pty and spdlog with nbin
code-asher Apr 9, 2019
e9c0c96
Label stderr/stdout from shared process
code-asher Apr 9, 2019
9de6400
Remove fork override
code-asher Apr 9, 2019
827dae6
Prevent "already disposed" errors when trying to kill disposed proxies
code-asher Apr 9, 2019
b05064e
Include spdlog dependencies
code-asher Apr 9, 2019
5bc5040
Shim /node_modules
code-asher Apr 9, 2019
8162e0f
Add node_modules to Docker ignore
code-asher Apr 10, 2019
cc92be0
Update nbin
code-asher Apr 15, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Label stderr/stdout from shared process
  • Loading branch information
@code-asher
code-asher committed Apr 9, 2019
commit e9c0c965bfde782b07271e87f6f4e17db7631577
6 changes: 3 additions & 3 deletions packages/server/src/vscode/sharedProcess.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { StdioIpcHandler } from "../ipc";
import { ParsedArgs } from "vs/platform/environment/common/environment";
import { Emitter } from "@coder/events/src";
import { retry } from "@coder/ide/src/retry";
import { logger, Level } from "@coder/logger";
import { logger, field, Level } from "@coder/logger";

export enum SharedProcessState {
Stopped,
Expand Down Expand Up @@ -127,13 +127,13 @@ export class SharedProcess {
activeProcess.on("exit", doReject);

activeProcess.stdout.on("data", (data) => {
logger.trace(data.toString());
logger.trace("stdout", field("data", data.toString()));
});

activeProcess.stderr.on("data", (data) => {
// Warn instead of error to prevent panic. It's unlikely stderr here is
// about anything critical to the functioning of the editor.
logger.warn(data.toString());
logger.warn("stderr", field("data", data.toString()));
});

this.ipcHandler = new StdioIpcHandler(activeProcess);
Expand Down