Skip to content
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

handle broken evaluation worker pipes more gracefully #277

Merged
merged 7 commits into from
Dec 10, 2023

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Dec 9, 2023

writeLine will throw a SysError exception, which obfuscates out-of-memory events where the eval worker is killed by the OS. readLine is suffering from the same problem and will be handled in a subsequent commit.

writeLine will throw a SysError exception, which obfuscates out-of-memory events where the eval worker is killed by the OS.
readLine is suffering from the same problem and will be handled in a subsequent commit.
@Mic92 Mic92 force-pushed the better-error-handling branch from b58e8c9 to 54a1df2 Compare December 9, 2023 16:37
we don't have toml files and deno has a smaller install closure.
@Mic92 Mic92 force-pushed the better-error-handling branch 5 times, most recently from f5b62e9 to e2e2395 Compare December 10, 2023 10:03
@Mic92 Mic92 force-pushed the better-error-handling branch from e2e2395 to 661313f Compare December 10, 2023 10:05
@Mic92
Copy link
Member Author

Mic92 commented Dec 10, 2023

@mergify queue

Copy link
Contributor

mergify bot commented Dec 10, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 6f4bee5

@mergify mergify bot merged commit 6f4bee5 into main Dec 10, 2023
13 checks passed
@Mic92 Mic92 deleted the better-error-handling branch December 10, 2023 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant