Skip to content
This repository has been archived by the owner on Jan 2, 2025. It is now read-only.

Changed several methods to return invisible(self) to allow chaining #21

Merged
merged 1 commit into from
Sep 21, 2018

Conversation

jayqi
Copy link
Contributor

@jayqi jayqi commented Sep 12, 2018

Enables the code shown in this example.

@codecov-io
Copy link

Codecov Report

Merging #21 into master will decrease coverage by 0.15%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
- Coverage   82.88%   82.72%   -0.16%     
==========================================
  Files          14       14              
  Lines         625      625              
==========================================
- Hits          518      517       -1     
- Misses        107      108       +1
Impacted Files Coverage Δ
R/workflow_dag.R 91.62% <100%> (-0.47%) ⬇️
R/module_package_function.R 84.44% <100%> (ø) ⬆️
R/module_custom_function.R 93.33% <100%> (ø) ⬆️

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 d7a9453...db8bb9b. Read the comment docs.

@jayqi
Copy link
Contributor Author

jayqi commented Sep 13, 2018

This contributes to #20

@jameslamb
Copy link
Collaborator

I just assigned @cwschultz88 to this PR

@cwschultz88 cwschultz88 merged commit d5a90e7 into uptake:master Sep 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants