Codecov Report

Merging#385 intomaster willnot change coverage.
The diff coverage isn/a.

@@           Coverage Diff           @@##           master     #385   +/-   ##=======================================  Coverage   90.76%   90.76%           =======================================  Files          50       50             Lines        6950     6950             Branches     1328     1328           =======================================  Hits         6308     6308             Misses        483      483             Partials      159      159

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatef7708c4...0dcbd68. Read thecomment docs.

I looked at the master builds for the last 10 rounds and the numbers are kind of all over the place and not consistent. So I'm not sure I see evidence that this will really improve the situation.

The specific change here is just reordering the environments which is benign, so I'm going to approve it and land it and we can see how future builds affect the hypothesis.

For reference, here's the last 10 passing master builds (685-711), latest first:

Ran for 20 min 45 sec
Ran for 26 min 58 sec
Ran for 22 min 12 sec
Ran for 24 min 32 sec
Ran for 17 min 33 sec
Ran for 19 min 5 sec
Ran for 23 min 14 sec
Ran for 16 min 15 sec
Ran for 17 min 58 sec
Ran for 14 min 58 sec