-
Notifications
You must be signed in to change notification settings - Fork 18
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
Draft for new 'Get started' vignette #63
Conversation
Codecov Report
@@ Coverage Diff @@
## master #63 +/- ##
=========================================
Coverage ? 63.54%
=========================================
Files ? 19
Lines ? 502
Branches ? 0
=========================================
Hits ? 319
Misses ? 183
Partials ? 0 Continue to review full report at Codecov.
|
Do you deploy the pkgdown website yourself, not by Travis? 😮 |
Reg vignettes vs README you might find the discussion in ropensci/software-review-meta#55 interesting (and don't hesitate to chime in there!) |
Thanks for the pointer, I'll take a look. 668a4bf happened automatically, but to preview it we'd need Netlify. I can't add this repo to Netlify, who could give Netlify the necessary permissions? |
Ah that'd be so cool, I was wishing it'd include a preview. Not sure, not me at least, will ask. |
- Explain deployment
- update advanced::running stages conditonally
- add info how to run tic locally
add info about R6 argument passing
- emphasize the goal of R code in tic.R
Merging now, let's continue discussion in #78. |
Addresses #57, not fully ready yet but merging anyway.