Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace TealData with teal_data in the docs #232

Closed
wants to merge 10 commits into from

Conversation

chlebowa
Copy link
Contributor

Closes #231

Modified example apps to use the following pattern:

  1. create cdisc_data
  2. use within to evaluate code one line at a time
  3. set datanames manually (enumerate datasets)
  4. set join keys automatically with default_cdisc_join_keys[datanames(data)]
  5. assign datasets in global
  6. create app
  7. run app

Added package prefixes to all teal* package functions.

@chlebowa chlebowa added the core label Nov 16, 2023
Copy link
Contributor

github-actions bot commented Nov 16, 2023

badge

Code Coverage Summary

Filename                    Stmts    Miss  Cover    Missing
------------------------  -------  ------  -------  --------------
R/tm_g_ae_oview.R             249     249  0.00%    83-365
R/tm_g_ae_sub.R               318     318  0.00%    59-413
R/tm_g_butterfly.R            387     387  0.00%    126-553
R/tm_g_decorate.R              46      46  0.00%    18-98
R/tm_g_events_term_id.R       298     298  0.00%    63-399
R/tm_g_heat_bygrade.R         320     320  0.00%    133-488
R/tm_g_patient_profile.R      710     710  0.00%    158-933
R/tm_g_spiderplot.R           323     323  0.00%    102-470
R/tm_g_swimlane.R             377     377  0.00%    125-559
R/tm_g_waterfall.R            424     424  0.00%    108-583
R/utils.R                      37      32  13.51%   32-91, 124-131
R/zzz.R                         1       1  0.00%    2
TOTAL                        3490    3485  0.14%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: e70fbc7

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Nov 16, 2023

Unit Tests Summary

  1 files    1 suites   0s ⏱️
  4 tests   4 ✔️ 0 💤 0
13 runs  13 ✔️ 0 💤 0

Results for commit 04b366c.

♻️ This comment has been updated with latest results.

R/tm_g_ae_oview.R Outdated Show resolved Hide resolved
R/tm_g_ae_oview.R Outdated Show resolved Hide resolved
R/tm_g_butterfly.R Outdated Show resolved Hide resolved
R/tm_g_patient_profile.R Outdated Show resolved Hide resolved
R/tm_g_spiderplot.R Outdated Show resolved Hide resolved
R/tm_g_waterfall.R Outdated Show resolved Hide resolved
@chlebowa
Copy link
Contributor Author

I'm afraid checks will keep failing because I neglected to put @78_... in my branch name.

@chlebowa chlebowa closed this Nov 16, 2023
@chlebowa chlebowa deleted the 231_TealData__to__teal_data@main branch November 16, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace TealData with teal_data in the docs
3 participants