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

31 update and add new vignette #34

Merged
merged 19 commits into from
Jan 14, 2025
Merged

Conversation

shajoezhu
Copy link
Collaborator

No description provided.

Copy link
Contributor

github-actions bot commented Jan 13, 2025

Unit Tests Summary

 1 files   2 suites   10s ⏱️
 7 tests  7 ✅ 0 💤 0 ❌
19 runs  19 ✅ 0 💤 0 ❌

Results for commit 9a27c5f.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@BFalquet BFalquet left a comment

Choose a reason for hiding this comment

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

Minor suggestions

vignettes/rtables.officer.Rmd Outdated Show resolved Hide resolved
vignettes/rtables.officer.Rmd Outdated Show resolved Hide resolved
vignettes/rtables.officer.Rmd Outdated Show resolved Hide resolved
Copy link
Contributor

github-actions bot commented Jan 14, 2025

✅ All contributors have signed the CLA
Posted by the CLA Assistant Lite bot.

@edelarua
Copy link
Contributor

I have read the CLA Document and I hereby sign the CLA

@edelarua
Copy link
Contributor

recheck

edelarua and others added 4 commits January 13, 2025 19:21
Signed-off-by: Emily de la Rua <[email protected]>
Signed-off-by: Emily de la Rua <[email protected]>
Signed-off-by: Emily de la Rua <[email protected]>
Copy link
Contributor

@edelarua edelarua left a comment

Choose a reason for hiding this comment

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

Hi @shajoezhu,

I reviewed your changes and all of the documentation throughout the package and made some small improvements (mostly just clarifying some of the wording), and I think it all looks great! This PR should be good to go now.

Thanks!!

@shajoezhu shajoezhu merged commit ff80321 into main Jan 14, 2025
31 checks passed
@shajoezhu shajoezhu deleted the 31_update_and_add_new_vignette branch January 14, 2025 01:56
@github-actions github-actions bot locked and limited conversation to collaborators Jan 14, 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.

3 participants