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

feat: allow prototype arguments #92

Merged
merged 8 commits into from
Dec 5, 2023
Merged

feat: allow prototype arguments #92

merged 8 commits into from
Dec 5, 2023

Conversation

sebffischer
Copy link
Member

Some objects have required construction arguments. When converting a dictionary to a data.table, these prototype arguments can be used for construction

Some objects have required construction arguments.
When converting a dictionary to a data.table, these prototype
arguments can be used for construction
R/Dictionary.R Outdated
assert_flag(.prototype)
args = list(...)
if (.prototype) {
args = insert_named(args, self$prototype_args(key))
Copy link
Member Author

Choose a reason for hiding this comment

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

change order

R/Dictionary.R Outdated
@@ -124,16 +133,22 @@ Dictionary = R6::R6Class("Dictionary",
#' @param ... (`any`)\cr
#' Passed down to constructor.
#'
#' @param required_args (`character()`).
#' @param required_args (`character()`)\cr
Copy link
Member Author

Choose a reason for hiding this comment

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

remove

@mllg
Copy link
Member

mllg commented Nov 29, 2023

LGTM

@mllg mllg requested a review from be-marc November 29, 2023 10:10
@sebffischer sebffischer merged commit 8344be9 into main Dec 5, 2023
3 checks passed
@sebffischer sebffischer deleted the feat/dict-prototype branch December 5, 2023 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants