Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

Review comments for 208 #219

Closed
wants to merge 4 commits into from
Closed

Review comments for 208 #219

wants to merge 4 commits into from

Conversation

rojer9-fb
Copy link
Contributor

Review comments for #208 that was closed prematurely

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 26, 2021
@codecov
Copy link

codecov bot commented Jan 26, 2021

Codecov Report

Merging #219 (d6c11a2) into master (66dfc32) will increase coverage by 0.15%.
The diff coverage is 90.76%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #219      +/-   ##
==========================================
+ Coverage   61.72%   61.87%   +0.15%     
==========================================
  Files          88       88              
  Lines        4345     4344       -1     
==========================================
+ Hits         2682     2688       +6     
+ Misses       1328     1324       -4     
+ Partials      335      332       -3     
Flag Coverage Δ
integration 56.65% <69.23%> (-0.01%) ⬇️
integration_storage 100.00% <ø> (ø)
unittests 49.82% <92.18%> (+0.65%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/cerrors/cerrors.go 12.50% <0.00%> (-12.50%) ⬇️
pkg/runner/test_runner.go 91.33% <92.18%> (+1.84%) ⬆️

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 66dfc32...d6c11a2. Read the comment docs.

@rojer9-fb
Copy link
Contributor Author

gentle ping @marcoguerri

@marcoguerri
Copy link
Contributor

marcoguerri commented Feb 3, 2021

Sorry for the latency, trying to put out some urgent fires internally. Will tentatively allocate some time by EOD to review this.

rojer9-fb added a commit that referenced this pull request Feb 26, 2021
With inter-step state serialization

This is take 2, incorporating a fix for #228 and comments from #219
rojer9-fb added a commit that referenced this pull request Feb 26, 2021
With inter-step state serialization

This is take 2, incorporating a fix for #228 and comments from #219
@rojer9-fb
Copy link
Contributor Author

this was reverted and being reintroduced in #232 , i incorporated all the changes there.

@rojer9-fb rojer9-fb closed this Feb 26, 2021
rojer9-fb added a commit that referenced this pull request Mar 5, 2021
With inter-step state serialization

This is take 2, incorporating a fix for #228 and comments from #219
rojer9-fb added a commit that referenced this pull request Mar 5, 2021
With inter-step state serialization

This is take 2, incorporating a fix for #228 and comments from #219
rojer9-fb added a commit that referenced this pull request Mar 10, 2021
With inter-step state serialization

This is take 2, incorporating a fix for #228 and comments from #219
xaionaro pushed a commit to xaionaro-facebook/contest that referenced this pull request Mar 12, 2021
With inter-step state serialization

This is take 2, incorporating a fix for facebookarchive#228 and comments from facebookarchive#219

Modified by: Dmitrii Okunev <[email protected]>
xaionaro pushed a commit to xaionaro-facebook/contest that referenced this pull request Mar 13, 2021
With inter-step state serialization

This is take 2, incorporating a fix for facebookarchive#228 and comments from facebookarchive#219
@rojer9-fb rojer9-fb deleted the runner2 branch March 25, 2021 12:21
llogen pushed a commit to 9elements/contest that referenced this pull request Aug 4, 2021
With inter-step state serialization

This is take 2, incorporating a fix for facebookarchive#228 and comments from facebookarchive#219
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants