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

flow_view_source_calls() only looks at the top level #157

Open
moodymudskipper opened this issue Jul 5, 2023 · 2 comments
Open

flow_view_source_calls() only looks at the top level #157

moodymudskipper opened this issue Jul 5, 2023 · 2 comments

Comments

@moodymudskipper
Copy link
Owner

So below it will only detect the first call:

source("a.R")
if (TRUE) {
  source("b.R")
}

I was probably too lazy to implement the recursion for my use case at the time.

@moodymudskipper
Copy link
Owner Author

Also is considering 1st arg as the path, which is probably right 99.5 % oof the time but we should use match.call and consider path

@pvictor
Copy link

pvictor commented Jul 6, 2023

Indeed I have some suppressWarnings(source("a.R")) that are ignored.

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

2 participants