Skip to content

Commit 01264e5

Browse files
fix things
1 parent 6d675ed commit 01264e5

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

inst/pre-commit-gha.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,11 @@ jobs:
4949
restore-keys: |
5050
precommit-${{env.UBUNTU_VERSION}}-renv-
5151
52-
- name: Cache pre-commit
53-
uses: actions/cache@v2
54-
with:
55-
path: ~/.cache/pre-commit
56-
key: pre-commit-${{ hashFiles('.pre-commit-config.yaml') }}-
52+
- name: Install R packages
53+
run: |
54+
if (!requireNamespace("renv", quietly = TRUE)) install.packages("renv")
55+
renv::restore()
56+
shell: Rscript {0}
5757

5858
- name: Run pre-commit
5959
uses: pre-commit/[email protected]

vignettes/ci.Rmd

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ to use the [roxygenize hook](https://lorenzwalthert.github.io/precommit/dev/arti
2828

2929
- by the creator of pre-commit. Certain CI settings in `.pre-commit-config.yaml`
3030
[are supported](https://pre-commit.ci).
31+
- actively developed.
3132
- very fast.
3233
- will at some point also be supported for repos hosted outside of GitHub.
3334
- No maintenance effort for you.
@@ -45,13 +46,15 @@ to use the [roxygenize hook](https://lorenzwalthert.github.io/precommit/dev/arti
4546
**Pros:**
4647

4748
- more flexible in how you run `pre-commit` if you need it.
48-
- out-of-the-box action is in [maintenance only mode](https://github.com/pre-commit/action)
4949
- No new authentication needed.
5050
- If your package has system dependencies and you want to use the roxygenize
5151
hook, you can install them.
5252

5353
**Cons:**
5454

55+
- out-of-the-box action is in
56+
[maintenance only mode](https://github.com/pre-commit/action) and less
57+
feature complete than [pre-commit.ci](https://pre-commit.ci).
5558
- You need to maintain the workflow file with a lot of boilerplate code and
5659
when GitHub changes the syntax, you need to adapt it to prevent failing
5760
builds.

0 commit comments

Comments
 (0)