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.
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
Ignore screen size error reported by ps #3634
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3634 +/- ##
==========================================
- Coverage 87.16% 86.92% -0.25%
==========================================
Files 59 59
Lines 1800 1805 +5
Branches 207 208 +1
==========================================
Hits 1569 1569
- Misses 176 181 +5
Partials 55 55
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hopefully this is the right place!
@@ -90,6 +90,16 @@ export async function getUnixChildProcessIds(pid: number): Promise<number[]> { | |||
}); | |||
|
|||
ps.on('error', reject); | |||
|
|||
ps.stderr.setEncoding('utf8'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know how this works with TS promises, but do you need to do something about the ps.on('error', reject);
on L92 as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think it will be a problem. Looking at the way vscode-node-debug handled error, it is effectively the same
Hooray - will try again when I get a chance. |
As suggested in microsoft/vscode#75932 (comment) we can ignore the screen size error reported on stderr.
May resolve #3580