Title: Support for R / renv as a language by lorenzwalthert · Pull Request #1799 · pre-commit/pre-commit · GitHub
Open Graph Title: Support for R / renv as a language by lorenzwalthert · Pull Request #1799 · pre-commit/pre-commit
X Title: Support for R / renv as a language by lorenzwalthert · Pull Request #1799 · pre-commit/pre-commit
Description: This PR aims to add 2nd level support for R to pre-commit via the virtual environment manager renv, motivated by isolated dependency management and (hopefully) compatibility with pre-commit.ci for 2nd level languages as outlined in lorenzwalthert/precommit#215. The implementation follows the recently contributed support for conda (#1232) for source code and tests as well as the implementation of the python virtualenv language. Further considerations: Initial tests only run on Linux. I have to add other platforms as we go if this PR is generally approved. I'd appreciate feedback to the questions in the source code comments, in particular on whether or not to remove in_env() as it seems not needed or whether it should be kept for consistency with other languages. Also, let me know if you want certain comments that explain stuff to be removed. Was not sure how much. If this gets through, we probably should add documentation for pre-commit.com. We might want to support file types for R (https://pre-commit.com/#filtering-files-with-types). Before this PR, I was not familiar with the pre-commit code base nor a very avid python user, hence sorry if there are obvious flaws in this PR.
Open Graph Description: This PR aims to add 2nd level support for R to pre-commit via the virtual environment manager renv, motivated by isolated dependency management and (hopefully) compatibility with pre-commit.ci for ...
X Description: This PR aims to add 2nd level support for R to pre-commit via the virtual environment manager renv, motivated by isolated dependency management and (hopefully) compatibility with pre-commit.ci for ...
Opengraph URL: https://github.com/pre-commit/pre-commit/pull/1799
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:6c55d8e1-7672-60a0-1f76-bb6416f24089 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | DB46:245004:2AF3105:3E1198D:6A5669FB |
| html-safe-nonce | 89ef53424c2a7e315838c4176ee0dff8263f697a2843a5e9d88ce035569db386 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEQjQ2OjI0NTAwNDoyQUYzMTA1OjNFMTE5OEQ6NkE1NjY5RkIiLCJ2aXNpdG9yX2lkIjoiNDE1MDYwOTczNjExMzc0NDM3OSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 4902db3d08fa5190e97f194ed71fdaeb8741b8cd1c7cfde9d726bd013c0d1bc3 |
| hovercard-subject-tag | pull_request:573689426 |
| github-keyboard-shortcuts | repository,commits,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/pre-commit/pre-commit/pull/1799/files |
| twitter:image | https://avatars.githubusercontent.com/u/10477073?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/10477073?s=400&v=4 |
| og:image:alt | This PR aims to add 2nd level support for R to pre-commit via the virtual environment manager renv, motivated by isolated dependency management and (hopefully) compatibility with pre-commit.ci for ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 1bfcd903bf3318822b10b18a6332fac40d3a21f7d94456dca766ab8fa7328068 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/pre-commit/pre-commit git https://github.com/pre-commit/pre-commit.git |
| octolytics-dimension-user_id | 6943086 |
| octolytics-dimension-user_login | pre-commit |
| octolytics-dimension-repository_id | 17689377 |
| octolytics-dimension-repository_nwo | pre-commit/pre-commit |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 17689377 |
| octolytics-dimension-repository_network_root_nwo | pre-commit/pre-commit |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 37f446b78ad53e6f593aa1a4243b87c76a3428ab |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width