You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The reason will be displayed to describe this comment to others. Learn more.
Was there a strong reason to delete it? I use make about as often as the equivalent littler script render.r. Could easily have modified line 8 here to use Rscript. Anyway ...
The reason will be displayed to describe this comment to others. Learn more.
Whoops, no, we can restore that. I saw the references to slides and figured it must have been something I had left over from another project -- I should have checked the git history first!
The reason will be displayed to describe this comment to others. Learn more.
I can live without it -- I often just call render.r -- but I think it would be nice for others, and it is after all a public repo. So will just revert the commit, ok?
The reason will be displayed to describe this comment to others. Learn more.
(And that just came up because in another project I had such a Makefile "to be helpful" and it just created confusion as it was attempted to make from RStudio ... but we cleared that up.)
6b697b7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there a strong reason to delete it? I use
make
about as often as the equivalent littler script render.r. Could easily have modified line 8 here to use Rscript. Anyway ...6b697b7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, no, we can restore that. I saw the references to slides and figured it must have been something I had left over from another project -- I should have checked the git history first!
6b697b7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can live without it -- I often just call
render.r
-- but I think it would be nice for others, and it is after all a public repo. So will just revert the commit, ok?6b697b7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, sounds good. thanks.
6b697b7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out that also restores a png file, so no. Will just re-commit with Rscript call instead of littler.
6b697b7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(And that just came up because in another project I had such a Makefile "to be helpful" and it just created confusion as it was attempted to
make
from RStudio ... but we cleared that up.)