-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
335 bring teal_slice(s)
to package index and create separate man pages for teal_slice(s)
utilities
#572
Conversation
Code Coverage Summary
Diff against main
Results for commit: fcf1166 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary 1 files 29 suites 23s ⏱️ Results for commit fcf1166. ♻️ This comment has been updated with latest results. |
I see some R CMD CHECK failures Missing link or links in documentation object 'teal_slices.Rd':
‘c.teal_slice’
See section 'Cross-references' in the 'Writing R Extensions' manual.
* checking for missing documentation entries ... OK
* checking for code/documentation mismatches ... OK
* checking Rd \usage sections ... WARNING
Documented arguments not in \usage in documentation object 'teal_slice':
‘x’ ‘show_all’ ‘trim_lines’
Documented arguments not in \usage in documentation object 'teal_slices':
‘x’ ‘i’ ‘recursive’ |
R CMD CHECK fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Please add entry to NEWS.md since this is user facing info.
Closes #335
teal_slice-utilities
is.teal_slice
,as.teal_slice
,as.list.teal_slice
,print.teal_slice
,format.teal_slice
teal_slice
teal_slice
@keywords internal
so does not appear in package indexteal_slice
man page@seealso
section to point to utility functions@description
of the pageDid the same for
teal_slices
and respective utility functions.Utility functions are not visible in package index, where
teal_slice
andteal_slices
were brought back to package index