Ignore renv folder by default

[DavZim]

I use flir in a project with 5 files that uses renv for version tracking.
When I use flir::lint() it recognizes 385 files, because all R files in renv/ of all dependencies are scanned as well.

I would expect the default behavior to ignore the renv folder. For now, I use flir::lint(exclude_path = "renv").

[etiennebacher]

Good idea, I have also implemented this behavior as default in Jarl. Do you want to make a PR?

[DavZim]

Would you just set the default paths in the R arguments?

[DavZim]

And are there any other folders that we should take into account and exclude?

I am happy to create a PR, just a couple of questions:

• do you want the version to be bumped + NEWS.md item filled?
• which functions are affected by this? Just lint and fix?

[etiennebacher]

I think I would internally take the union of exclude_paths and a hardcoded list of paths to ignore. See here for this list of paths to ignore by default: https://github.com/etiennebacher/jarl/blob/015f7ad1c93d2dd38b9ec05c52776b4743d98442/crates/jarl-core/src/discovery.rs#L21

The idea is that we don't want those paths to be analyzed if the user passes their own exclude_paths, e.g. I don't renv to be analyzed just because I did lint(exclude_paths = "my/custom/path.R").

[etiennebacher]

The version should be bumped and NEWS should be updated.

This should affect the fix_*() and lint_*() functions only, yes. Feel free to get started with an incomplete PR if you prefer and then we can discuss the details there.

[DavZim]

Ill try to have a look at it later!