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

Temporary files / chart files at top-level? #54

Open
eddelbuettel opened this issue Feb 20, 2017 · 0 comments
Open

Temporary files / chart files at top-level? #54

eddelbuettel opened this issue Feb 20, 2017 · 0 comments

Comments

@eddelbuettel
Copy link
Collaborator

eddelbuettel commented Feb 20, 2017

There are quite a few Hadley-isms I don't quite understand, and doubling up a README at GitHub as both an .Rmd and and .md is one. I guess if one thinks one has to re-run the README many times then having a full blown knitr document may make sense. But I digress.

In the knitr document you have

fig.path = "README-"

Wouldn't it be a little neater if you made it

fig.path = "images/README-"

to get them out of the way at the top-level? Just a thought, feel free to close at sight unseen :)

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

No branches or pull requests

1 participant