Skip to content
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

Add inline-reporting vignette #25

Merged
merged 7 commits into from
Apr 24, 2022
Merged

Add inline-reporting vignette #25

merged 7 commits into from
Apr 24, 2022

Conversation

gadenbuie
Copy link
Owner

@gadenbuie gadenbuie commented Apr 21, 2022

Hi @tjmahr! I used (i.e. heavily borrowed) from your awesome lists with knitr blog post for a vignette for epoxy. I'm tagging you into this PR hoping you'll let me know if you mind me re-using your material a bit. If there's anything you'd like me to change, please let me know!

Here's a link where you can preview the article: http://pkg.garrickadenbuie.com/epoxy/preview/pr25/articles/inline-reporting.html

(To be honest, when I started out I fully intended to simply re-write your post using epoxy instead of inline R chunks. But I quickly ran into the problem that you used packages I don't want to add to the dependencies of this package, which derailed my original plan. Oh well, I think it worked out for the best.)

@gadenbuie gadenbuie marked this pull request as ready for review April 21, 2022 03:45
@tjmahr
Copy link

tjmahr commented Apr 23, 2022

This is great, go for it. Just a note that I don’t use emo::ji() since I started using R 4.2.0. That was what I used to have to get emoji into blogposts.

@gadenbuie gadenbuie merged commit 7adbea8 into main Apr 24, 2022
@gadenbuie gadenbuie deleted the vignette/inline-reporting branch April 24, 2022 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants