Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Change open_browser to open browser in child process #1495

Merged
merged 2 commits into from
Aug 12, 2020

Conversation

jspspike
Copy link
Contributor

Closes #516

Seems to fix the problem on linux, probably should get tested on mac os and windows before being merged.

@jspspike jspspike requested a review from a team August 11, 2020 20:29
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me!!

@EverlastingBugstopper EverlastingBugstopper added this to the 1.11.0 milestone Aug 12, 2020
@EverlastingBugstopper EverlastingBugstopper merged commit ad32c7b into master Aug 12, 2020
@jspspike jspspike deleted the josh/child-browser branch August 12, 2020 16:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

If wrangler preview starts the first browser process, it hangs waiting for it to exit
2 participants