-
Notifications
You must be signed in to change notification settings - Fork 129
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
As a dev, I want to create template files for R6 classes [FR] #1008
Comments
Works exactly the same as add_fct() and add_utils(). - add argument 'r6' to add_module() - add add_r6() helper function calling add_r_files with slightly different parameters - adjust append_roxygen_comment() to create R6 template - add man files and NAMESPACE exports
Works exactly the same as add_fct() and add_utils(). - add argument 'r6' to add_module() - add add_r6() helper function calling add_r_files with slightly different parameters - adjust append_roxygen_comment() to create R6 template - add man files and NAMESPACE exports
That sounds basically like TidyModules |
hi @D3SL I think tidymodules is more of a framework that suggests a specific style for usining R6 and (adjusted) modules when developing shiny apps. Sometimes using basic R6 classes (without a fully fledged framework) can have merit on its own. For example, this is suggested in https://engineering-shiny.org/common-app-caveats.html?q=r6#using-r6-as-data-storage and https://youtu.be/fo4eiwGE6bw as a useful development pattern in some cases. Thus, this FR only asks for a quick boilerplate generator in the style of the already available |
Quick helper in the spirit of add_XXX
Create R6 boilerplate in a new file
Describe the solution you'd like
add_r6()
with same interface asadd_fct()
andadd_utils
R6
comes with shiny and is thus already a dependency for golem, so the feature does not need "real" additional dependencies.The overall usefulness though is difficult to asses for me, it would certainly be beneficial when creating R6 classes for exchanging data between modules.
The text was updated successfully, but these errors were encountered: