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

81 cran #84

Merged
merged 6 commits into from
Nov 26, 2024
Merged

81 cran #84

merged 6 commits into from
Nov 26, 2024

Conversation

shajoezhu
Copy link
Collaborator

Pull Request

Fixes #nnn

@shajoezhu shajoezhu added the sme label Nov 19, 2024
Copy link
Contributor

github-actions bot commented Nov 19, 2024

badge

Code Coverage Summary

Filename             Stmts    Miss  Cover    Missing
-----------------  -------  ------  -------  ---------
R/tabulate_rbmi.R       82       0  100.00%
TOTAL                   82       0  100.00%

Diff against main

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

Results for commit: 0ab99d4

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Nov 19, 2024

Unit Tests Summary

7 tests   6 ✅  0s ⏱️
3 suites  1 💤
1 files    0 ❌

Results for commit 0ab99d4.

♻️ This comment has been updated with latest results.

@shajoezhu

This comment was marked as outdated.

@shajoezhu
Copy link
Collaborator Author

shajoezhu commented Nov 20, 2024

Please also use single quotes in your title of the DESCRIPTION.

The Description field is intended to be a (one paragraph) description of
what the package does and why it may be useful. Please add more details
about the package functionality and implemented methods in your
Description text.

Please add \value to .Rd files regarding exported methods and explain
the functions results in the documentation. Please write about the
structure of the output (class) and also what the output means. (If a
function does not return a value, please document that too, e.g.
\value{No return value, called for side effects} or similar)
Missing Rd-tags:
a_rbmi_lsmeans.Rd: \value
h_tidy_pool.Rd: \value
s_rbmi_lsmeans.Rd: \value
summarize_rbmi.Rd: \value

  • update Description
  • a_rbmi_lsmeans.Rd: \value
  • h_tidy_pool.Rd: \value
  • s_rbmi_lsmeans.Rd: \value
  • summarize_rbmi.Rd: \value

@shajoezhu
Copy link
Collaborator Author

@shajoezhu shajoezhu enabled auto-merge (squash) November 26, 2024 14:21
Copy link

@ayogasekaram ayogasekaram left a comment

Choose a reason for hiding this comment

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

lgtm!

@shajoezhu shajoezhu merged commit aebccd8 into main Nov 26, 2024
29 checks passed
@shajoezhu shajoezhu deleted the 81_cran branch November 26, 2024 14:27
@github-actions github-actions bot locked and limited conversation to collaborators Nov 26, 2024
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.

2 participants