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

Local result dir #523

Merged
merged 1 commit into from
Mar 18, 2019
Merged

Local result dir #523

merged 1 commit into from
Mar 18, 2019

Conversation

bootstraponline
Copy link
Contributor

@bootstraponline bootstraponline commented Mar 15, 2019

Fix #500

@codecov-io
Copy link

codecov-io commented Mar 15, 2019

Codecov Report

Merging #523 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #523   +/-   ##
=========================================
  Coverage     80.03%   80.03%           
  Complexity      768      768           
=========================================
  Files            75       75           
  Lines          2314     2314           
  Branches        341      341           
=========================================
  Hits           1852     1852           
  Misses          295      295           
  Partials        167      167

@miguelslemos
Copy link
Contributor

This PR contains commits related to #456. But local result dir implementation looks good to me.

@bootstraponline bootstraponline changed the base branch from master to test_retry March 16, 2019 15:51
@bootstraponline
Copy link
Contributor Author

I updated to base branch to avoid the unrelated commits.

@bootstraponline bootstraponline changed the base branch from test_retry to master March 18, 2019 14:25
Use exit code 3 on fatal error instead of -1
@bootstraponline bootstraponline merged commit 3b2a2ba into master Mar 18, 2019
@bootstraponline bootstraponline deleted the local-result-dir branch March 18, 2019 14:44
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.

3 participants