Skip to content

Conversation

@oesteban
Copy link
Contributor

Close #2250

@satra
Copy link
Member

satra commented Oct 24, 2017

lgtm - i was hoping to leave this on for a new comer :) but i'm sure we can find other things.

@oesteban
Copy link
Contributor Author

sorry about that

@codecov-io
Copy link

codecov-io commented Oct 24, 2017

Codecov Report

Merging #2251 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2251      +/-   ##
==========================================
+ Coverage   72.32%   72.32%   +<.01%     
==========================================
  Files        1183     1183              
  Lines       59008    59008              
  Branches     8490     8490              
==========================================
+ Hits        42675    42679       +4     
+ Misses      14949    14945       -4     
  Partials     1384     1384
Flag Coverage Δ
#smoketests 72.32% <100%> (ø) ⬆️
#unittests 69.9% <100%> (ø) ⬆️
Impacted Files Coverage Δ
nipype/pipeline/engine/nodes.py 83.78% <100%> (ø) ⬆️
nipype/utils/profiler.py 41.56% <0%> (+1.2%) ⬆️
nipype/pipeline/plugins/multiproc.py 78.16% <0%> (+1.4%) ⬆️

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 20b113c...eaa1bd0. Read the comment docs.

@mgxd mgxd added this to the 0.14.0 milestone Oct 24, 2017
@mgxd mgxd merged commit 37c9db6 into master Oct 24, 2017
@oesteban oesteban deleted the oesteban-patch-2 branch October 24, 2017 22:59
@satra satra modified the milestone: 0.14.0 Oct 28, 2017
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.

5 participants