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

Set outputStrategy for run scope #5019

Merged
merged 1 commit into from
Oct 5, 2023
Merged

Set outputStrategy for run scope #5019

merged 1 commit into from
Oct 5, 2023

Conversation

RustedBones
Copy link
Contributor

As we fork the JVM, set the outputStrategy to StdoutOutput for run scope so we can get the repl working and stacktraces in runMain

@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #5019 (0b91348) into main (e23cc63) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

❗ Current head 0b91348 differs from pull request most recent head c0bb4d0. Consider uploading reports for the commit c0bb4d0 to get more accurate results

@@           Coverage Diff           @@
##             main    #5019   +/-   ##
=======================================
  Coverage   63.17%   63.17%           
=======================================
  Files         283      283           
  Lines       10676    10676           
  Branches      766      766           
=======================================
  Hits         6745     6745           
  Misses       3931     3931           

@RustedBones RustedBones merged commit ab93b71 into main Oct 5, 2023
11 checks passed
@RustedBones RustedBones deleted the output-strategy branch October 5, 2023 14:38
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.

2 participants