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

fix: Fix a bunch of race conditions and replace fs with gensync equivalent #6060

Merged
merged 10 commits into from
Aug 23, 2021

Conversation

DeMoorJasper
Copy link
Member

@DeMoorJasper DeMoorJasper commented Aug 20, 2021

What kind of change does this PR introduce?

More preparation work to replace the resolver, this PR resolves a bunch of "hidden" race conditions in the vue 2 transpiler and replaces the readFile and isFile utils with their gensync equivalent

This should be the last PR before the big one to replace the resolver

@lbogdan
Copy link
Contributor

lbogdan commented Aug 20, 2021

Build for latest commit 615672d is at https://pr6060.build.csb.dev/s/new.

@lbogdan lbogdan temporarily deployed to pr6060 August 20, 2021 14:36 Inactive
@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 20, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 615672d:

Sandbox Source
Notifications Test Configuration

@lbogdan lbogdan temporarily deployed to pr6060 August 20, 2021 17:13 Inactive
@lbogdan lbogdan temporarily deployed to pr6060 August 23, 2021 08:02 Inactive
@lbogdan lbogdan temporarily deployed to pr6060 August 23, 2021 08:54 Inactive
@lbogdan lbogdan temporarily deployed to pr6060 August 23, 2021 09:44 Inactive
@lbogdan lbogdan temporarily deployed to pr6060 August 23, 2021 10:02 Inactive
@lbogdan lbogdan temporarily deployed to pr6060 August 23, 2021 11:49 Inactive
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