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 #23

Merged
merged 27 commits into from
Oct 7, 2024
Merged
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
a590a6e
Adding the pre-commit hook
gvegayon Oct 7, 2024
e241581
Removing adding README.md back to pre-commit
gvegayon Oct 7, 2024
af1ea4c
Adding pre-commit to Rbuildignore
gvegayon Oct 7, 2024
6807d9d
Adding pre-commit as dependency
gvegayon Oct 7, 2024
b6bd23e
Merge branch 'main' into pre-commit
gvegayon Oct 7, 2024
6610a22
Passing Rscript differently
gvegayon Oct 7, 2024
7abdeac
Merge branch 'pre-commit' of github.com:UofUEpiBio/epiworldR into pre…
gvegayon Oct 7, 2024
9c78ebc
Get to get the library
gvegayon Oct 7, 2024
0146a6e
Setting up R in pre-commit
gvegayon Oct 7, 2024
8016f59
Setting up where to install the package and from where
gvegayon Oct 7, 2024
cfb6543
Both can only be used in mac and windows
gvegayon Oct 7, 2024
b442dd0
Adding jammy to the repo
gvegayon Oct 7, 2024
b745b2c
Removing dangling pre-commit file
gvegayon Oct 7, 2024
f3e9f0c
Setting no cache root
gvegayon Oct 7, 2024
7c29b84
Reverting root cache pre-commit
gvegayon Oct 7, 2024
060dcdb
Running pre-commit alternatively
gvegayon Oct 7, 2024
1814563
Update pre-commit.yaml
gvegayon Oct 7, 2024
80b88a2
Correct hooks
gvegayon Oct 7, 2024
e916105
Merge branch 'pre-commit' of github.com:UofUEpiBio/epiworldR into pre…
gvegayon Oct 7, 2024
1adc21b
Forgot to install it:
gvegayon Oct 7, 2024
4cd89fb
Dealing with styler
gvegayon Oct 7, 2024
762c060
Revising how to deal with pre-commit
gvegayon Oct 7, 2024
ab60294
Forcing the R version to be 4.4.1 in precommit R
gvegayon Oct 7, 2024
1acee27
Adding some contextual information
gvegayon Oct 7, 2024
080368b
Reverting how pre-commit works
gvegayon Oct 7, 2024
79c66d9
Adding pre-commit Renv
gvegayon Oct 7, 2024
654616c
Need a cache-root maybe
gvegayon Oct 7, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Dealing with styler
  • Loading branch information
gvegayon committed Oct 7, 2024
commit 4cd89fb49994fd59d1ce904c62b3631905bd1065
14 changes: 7 additions & 7 deletions .github/workflows/pre-commit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ jobs:
- name: Set up Python
uses: actions/setup-python@v3

- name: Set up R
uses: r-lib/actions/setup-r@v2
# - name: Set up R
# uses: r-lib/actions/setup-r@v2

- name: Cache pre-commit
uses: actions/cache@v4
Expand All @@ -41,11 +41,11 @@ jobs:
pre-commit install --install-hooks
shell: bash

- name: Install styler
run: |
options(repos = c(CRAN = "https://packagemanager.posit.co/cran/__linux__/jammy/latest"))
install.packages("styler")
shell: Rscript {0}
# - name: Install styler
# run: |
# options(repos = c(CRAN = "https://packagemanager.posit.co/cran/__linux__/jammy/latest"))
# install.packages("styler")
# shell: Rscript {0}

- name: Run pre-commit hooks
run: pre-commit run --all-files
Loading