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

pre-commit action #2

Open
gvegayon opened this issue Oct 7, 2024 · 0 comments
Open

pre-commit action #2

gvegayon opened this issue Oct 7, 2024 · 0 comments
Labels
new action Proposing a new action

Comments

@gvegayon
Copy link
Member

gvegayon commented Oct 7, 2024

Pre-commit is used across CFA, yet in many places, it is used in a sort of hacky way. The reason: Renv and all the caching going around that the pre-commit/actions action can't deal with (here is an example). Having this more standardize would be nice! A couple of gotchas I've experienced:

  • The R pre-commit action (here) uses renv. Currently with version R 4.4.1. If anything happens in the runner, this version breaks.
  • Cacheing sort of happens with the pre-commit/action, but only for stuff under the ~/.cache/pre-commit folder. In the case of R, it is mostly symlinks that tend to break.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new action Proposing a new action
Projects
None yet
Development

No branches or pull requests

1 participant