Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

use_addins() #353

Closed
EmilHvitfeldt opened this issue May 21, 2018 · 3 comments
Closed

use_addins() #353

EmilHvitfeldt opened this issue May 21, 2018 · 3 comments
Labels
feature a feature request or enhancement help wanted ❤️ we'd love your help!

Comments

@EmilHvitfeldt
Copy link
Member

Function to help setup the necessary steps for addins in a package.

Would

  • Create a file at inst/rstudio/addins.dcf (if not already present)
  • Add the 4 lines to file and populates the binding.

Example

use_addins("addin_function")

(following lines added in "inst/rstudio/addins.dcf")
Name: --- please supply name ---
Description: --- please supply description ---
Binding: addin_function
Interactive: false
@moodymudskipper
Copy link

I came to ask for almost exactly that, except that I'd suggest :

use_addin <- function(binding, name = binding, description ="", interactive = FALSE)`

And overwrite an addin specification that would have the same name or the same binding function.

@hadley hadley added feature a feature request or enhancement help wanted ❤️ we'd love your help! labels Nov 30, 2018
@hadley
Copy link
Member

hadley commented Nov 30, 2018

We'd love to see a PR for this. I think it should be fairly simple, following a template from an existing function.

@apreshill
Copy link
Contributor

apreshill commented Jan 19, 2019 •
edited
Loading

Tagging @haozhu233 - TBD PR based on his awesome rstudio::conf 2019 talk: https://github.com/hebrewseniorlife/addin_demo

haozhu233 added a commit to haozhu233/usethis that referenced this issue Jan 21, 2019
@haozhu233
Added use_addin as requested in r-lib#353
b8a2c2b
@haozhu233 haozhu233 mentioned this issue Jan 21, 2019
Merged
@hadley hadley closed this as completed in fbb03d1 Feb 4, 2019
jennybc pushed a commit that referenced this issue Mar 6, 2019
@haozhu233 @jennybc
Move proj_path to a more appropriate place in use_addin (#619)
a39e54b 
* Added use_addin as requested in #353

* remove return inside invisible

* Improved tests for use_addin; Added item to NEWS

* Following Hadley's comment, put in addin variable through render_template

* make the newline part a little more dynamic

* Tweak news bullet

* Move proj_path up as Jenny suggested
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
feature a feature request or enhancement help wanted ❤️ we'd love your help!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@hadley @apreshill @EmilHvitfeldt @moodymudskipper