Skip to content
This repository has been archived by the owner on Mar 27, 2023. It is now read-only.

refactor ETL #233

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

refactor ETL #233

wants to merge 1 commit into from

Conversation

maxheld83
Copy link
Contributor

No description provided.

@maxheld83 maxheld83 added this to To do in Scholcomm Analytics Kanban via automation Jun 10, 2020
@maxheld83 maxheld83 added the ETL extract, transform, load label Jun 10, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jun 10, 2020

Codecov Report

Merging #233 into master will decrease coverage by 7.22%.
The diff coverage is 14.89%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #233      +/-   ##
==========================================
- Coverage   34.56%   27.34%   -7.23%     
==========================================
  Files           4        5       +1     
  Lines          81      128      +47     
==========================================
+ Hits           28       35       +7     
- Misses         53       93      +40     
Impacted Files Coverage Δ
R/helpers.R 27.77% <0.00%> (-27.78%) ⬇️
R/modules.R 0.00% <0.00%> (ø)
R/data.R 81.08% <100.00%> (+4.41%) ⬆️

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 b832ef6...0e50c6f. Read the comment docs.

@maxheld83 maxheld83 self-assigned this Jun 10, 2020
@maxheld83
Copy link
Contributor Author

@njahn82 just to make sure you're not wasting time with this; this is currently a total mess, and I haven't really figured out how I want to go about this.
Probably not worth it to try and make sense of this PR as it is now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ETL extract, transform, load
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants