-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support for .Rmd, .Rpres & .Rnw #98
Comments
How critical is it that we support beyond .R and .Rmd/.Rpres? |
Probably not too critical. Maybe just work in support for md/pres and then come back to Rnw if requested? As far as I know, I've never come across a Rnw in the wild. |
Agreed
…On Tue, May 14, 2019, 9:45 AM Adam Spannbauer ***@***.***> wrote:
Probably not too critical. Maybe just work in support for md/pres and then
come back to Rnw if requested? As far as I know, I've never come across a
Rnw in the wild.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#98?email_source=notifications&email_token=AD4VHGURCCV2G3LPU4Z575LPVK655A5CNFSM4HMZEFJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVLQU5Y#issuecomment-492243575>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AD4VHGT75P467OM2MQWKCBTPVK655ANCNFSM4HMZEFJA>
.
|
@AdamSpannbauer this was addressed in #105 right? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Related to #95
To support file extensions outside of
.R
well some additional logic will be needed. If we choose to mirror the support packrat has then this includes adding support for.Rmd
/.Rpres
&.Rnw
.I'm not familiar with
.Rnw
tbh. packrat handles it with their helperpackrat:::fileDependencies.Rnw
which usesutils::Stangle
then runs theirR
dependency finder on the output.Below is a potential start to a string based dependency finder for
.Rmd
/.Rpres
Contents of "test.Rmd":
The text was updated successfully, but these errors were encountered: