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

[now dev] Fix directory listing race condition #2591

Merged
merged 18 commits into from
Jul 22, 2019

Conversation

AndyBitz
Copy link
Contributor

Fix directory listing race condition

@codecov-io
Copy link

Codecov Report

Merging #2591 into canary will increase coverage by 0.01%.
The diff coverage is 28.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           canary    #2591      +/-   ##
==========================================
+ Coverage   13.82%   13.84%   +0.01%     
==========================================
  Files         257      257              
  Lines        9357     9359       +2     
  Branches     1058     1058              
==========================================
+ Hits         1294     1296       +2     
  Misses       7941     7941              
  Partials      122      122
Impacted Files Coverage Δ
src/util/dev/builder.ts 74.69% <100%> (+0.15%) ⬆️
src/util/dev/server.ts 64.44% <16.66%> (+0.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c61502e...6159d65. Read the comment docs.

@AndyBitz AndyBitz merged commit 6e3a495 into canary Jul 22, 2019
@AndyBitz AndyBitz deleted the now-dev/fix-different-file-usage branch July 22, 2019 16:34
AndyBitz added a commit that referenced this pull request Jul 25, 2019
* [now dev] Fix directory listing race condition

* Resolve file names

* Check only matched files

* Log stdout and stderr

* Log expected

* Change test

* Revert "Change test"

This reverts commit 149723b.

* Add timeout to request

* Adjust retries

* Don't use unref

* Remove tests

* Revert "Remove tests"

This reverts commit 6159d65.

* Adjusted test

* Make sure directory exists

* Fix tests

* Add yarn.lock

* Adjust timeout
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.

None yet

3 participants