-
Notifications
You must be signed in to change notification settings - Fork 33
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
Shorten generation_time_opts()
to gt_opts()
as alias
#698
Conversation
CI errors will be fixed when #700 is merged. |
Leaving a note here that |
This has been resolved but was one of two errors in the CI. |
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
Co-authored-by: Sebastian Funk <[email protected]>
Co-authored-by: Sebastian Funk <[email protected]>
Description
This PR closes #564.
The
generation_time_opts()
function was deemed too long and tedious to specify so has been shortened togt_opts()
. Calls to both functions are equivalent and they share the same documentation.Initial submission checklist
devtools::test()
anddevtools::check()
).devtools::document()
).lintr::lint_package()
).After the initial Pull Request