Skip to content

Improves random data documentation (rAD**)#376

Merged
averissimo merged 11 commits into
mainfrom
367-update_data_vignettes
Aug 8, 2025
Merged

Improves random data documentation (rAD**)#376
averissimo merged 11 commits into
mainfrom
367-update_data_vignettes

Conversation

@averissimo

@averissimo averissimo commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

Pull Request

Changes description

  • 1 single man page for all data objects
  • Adds "Format" section generated automatically from data.
  • Minor lint cleanup in data-raw/data.R
  • Consistent naming of datasets with random.cdisc.data package.
  • Data objects no longer have @keywords internal and are build in site

@averissimo averissimo changed the title Builds data references in site Improves random data documentation _(rAD**)_ Jul 25, 2025
@averissimo averissimo changed the title Improves random data documentation _(rAD**)_ Improves random data documentation (rAD**) Jul 25, 2025
Comment thread DESCRIPTION
@averissimo
averissimo marked this pull request as ready for review July 29, 2025 12:47
@github-actions

github-actions Bot commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   15 suites   2s ⏱️
163 tests 163 ✅ 0 💤 0 ❌
218 runs  218 ✅ 0 💤 0 ❌

Results for commit 7c6c70d.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 29, 2025

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  -------------------
R/cdisc_data.R                   1       0  100.00%
R/deprecated.R                  18       0  100.00%
R/dummy_function.R               2       2  0.00%    15-16
R/formatters_var_labels.R       61       0  100.00%
R/join_key.R                    38       0  100.00%
R/join_keys-c.R                 12       0  100.00%
R/join_keys-extract.R          128       0  100.00%
R/join_keys-names.R             15       0  100.00%
R/join_keys-parents.R           30       0  100.00%
R/join_keys-print.R             45       0  100.00%
R/join_keys-utils.R             73       3  95.89%   35-38
R/join_keys.R                   21       0  100.00%
R/teal_data-class.R             36       6  83.33%   50-52, 59, 104-105
R/teal_data-constructor.R       11       2  81.82%   49, 52
R/teal_data-extract.R            3       0  100.00%
R/teal_data-get_code.R          13       8  38.46%   114-120, 124
R/teal_data-names.R              8       1  87.50%   42
R/teal_data-show.R               5       5  0.00%    14-20
R/topological_sort.R            32       0  100.00%
R/verify.R                      42      11  73.81%   67, 97-101, 104-108
TOTAL                          594      38  93.60%

Diff against main

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

Results for commit: 7c6c70d

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@donyunardi donyunardi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just have a small update request, but the rest looks good.

Comment thread R/data.R Outdated
averissimo and others added 4 commits August 8, 2025 14:25
Co-authored-by: Dony Unardi <donyunardi@gmail.com>
Signed-off-by: André Veríssimo <211358+averissimo@users.noreply.github.com>
@averissimo
averissimo merged commit 4ce75a6 into main Aug 8, 2025
29 checks passed
@averissimo
averissimo deleted the 367-update_data_vignettes branch August 8, 2025 12:45
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Question]: what is the strategy for visibility of datasets

2 participants