uptake / updraft

R package for building flexible workflows
Other
13 stars 12 forks source link

added invisible to return(NULL) #15

Closed jem12f closed 6 years ago

jem12f commented 6 years ago

Hey,

Hope your day is going well! I worked with James L on this so you know it's legit. Please accept my pull request. This addresses #7

Thanks!

codecov-io commented 6 years ago

Codecov Report

Merging #15 into master will decrease coverage by 0.32%. The diff coverage is 81.81%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage   82.62%   82.29%   -0.33%     
==========================================
  Files          14       14              
  Lines         610      610              
==========================================
- Hits          504      502       -2     
- Misses        106      108       +2
Impacted Files Coverage Δ
R/connection_directed.R 96.77% <0%> (ø) :arrow_up:
R/utils_modules.R 49.09% <0%> (ø) :arrow_up:
R/utils_check_interface.R 60% <100%> (ø) :arrow_up:
R/utils_execution.R 92.85% <100%> (ø) :arrow_up:
R/workflow_dag.R 90.86% <100%> (-0.97%) :arrow_down:
R/utils_connections.R 88.31% <100%> (ø) :arrow_up:
R/module_package_function.R 84.44% <100%> (ø) :arrow_up:
R/module_custom_function.R 93.33% <100%> (ø) :arrow_up:

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 4a85034...9f4844a. Read the comment docs.

cwschultz88 commented 6 years ago

Thanks, everything looks great!