Closed hutch3232 closed 2 years ago
Looks great!
And yes, the test revealed nicely that R is smart enough about parameter parsing that even objects with the same names as parameters work fine, with or without .character_only=TRUE
. That is great news!
This is now in the dev branch (manuallly merged), so I'm closing the PR.
Thanks for the contribution! (and holler if anything seems missing)
@torfason
Note that while I provided warning in several places that people should try to stay away from using argument names in their hidden objects, my testing (tests) seemed to show that this works.