-
Notifications
You must be signed in to change notification settings - Fork 8
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
Remove requirement for Rstudio #64
Comments
I'm pretty agnostic about it. I tend to use RStudio for most things anyway, but the way I see it flexibility is good so if you can do away with the dependency that's a nice bonus. It wasn't an issue that I had to use RStudio, it was an issue that this wasn't documented. |
Thanks for the feedback @adamhsparks. I'll keep this as an enhancement issue but perhaps not a major priority. |
I just got bit by this again, I've really moved away from using RStudio by default. Has any more thought been put into this? |
Looking through the code for another issue that I was having, I see why it's required now. It would be nice not to need it, but I understand why it's there. |
@adamhsparks why is it required? I think projects could be created and then if no RStudio, they wouldn't get opened but you'd get a path to where they are? |
I was seeing the interactive notebook as necessary to use in RStudio, but maybe it's not? I mean, you could use it without the project or the interactive notebook, but they seemed to be part and parcel of the package and I just didn't want to make extra work for someone. |
is the interactivity of the notebook important for you or would you be ok knitting to another format? |
It's not important to me at all, I don't use it TBH. I was thinking for other users. |
see #86 , I didn't use the index either. |
Related to #11 and ropensci/dev_guide#183.
I wonder if we could give the option to easily do away with rstudio dependency by
purl
ing theindex.Rmd
into an.R
script. @adamhsparks, would a testing workflow in a script be better for you?The text was updated successfully, but these errors were encountered: