Skip to content

Fix: bootstrap race and OS check - #717

Open
roccomuso wants to merge 2 commits into
RabbyHub:publish/prodfrom
roccomuso:fix/dev-bootstrap-race-and-os-check
Open

roccomuso wants to merge 2 commits into
RabbyHub:publish/prodfrom
roccomuso:fix/dev-bootstrap-race-and-os-check

Conversation

@roccomuso

Copy link
Copy Markdown

Closes: #716

Fixes two dev-environment papercuts hit while bootstrapping local dev: a build-timing race that made Electron try to load assets/desktop_shell before webpack finished writing it (now waits on the compiler's done hook instead of a fixed 15s guess), and a 3-year-old 99 debug sentinel that made the "unsupported OS" modal fire for every developer regardless of actual OS version.

@roccomuso

Copy link
Copy Markdown
Author

cc @richardo2016x

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.

Your operating system is not supported.

1 participant