Skip to content

Commit 67f4fdf

Browse files
author
Susan Vanderplas
committed
cleaning out crud that isn't necessary
1 parent 0be50b0 commit 67f4fdf

File tree

1 file changed

+5
-21
lines changed

1 file changed

+5
-21
lines changed

.github/workflows/main.yml

+5-21
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,6 @@ jobs:
1515
steps:
1616
- uses: actions/checkout@v4
1717

18-
- name: Change ownership of /github/home
19-
run: sudo chown -R $(whoami) /github/home
20-
2118
- name: Install system dependencies
2219
if: runner.os == 'Linux'
2320
run: |
@@ -32,22 +29,6 @@ jobs:
3229
&& sudo apt install gh -y
3330
shell: /bin/bash {0}
3431

35-
#
36-
# - name: Setup Python and virtual environment using requirements file
37-
# id: setup-python-venv
38-
# uses: srvanderplas/[email protected]
39-
# with:
40-
# python-version-file: '.python-version'
41-
# cache-dependency-path: requirements.txt
42-
# install-cmd: "./py-deps -i"
43-
44-
# - name: Test
45-
# run: |
46-
# echo $RETICULATE_PYTHON;
47-
# echo $RETICULATE_PYTHON_ENV;
48-
# echo "$pythonLocation" >> $GITHUB_PATH;
49-
#
50-
5132
- name: Renv setup
5233
uses: r-lib/actions/setup-renv@v2
5334

@@ -67,15 +48,18 @@ jobs:
6748
reticulate::virtualenv_create("venv", requirements="requirements.txt")
6849
reticulate::use_virtualenv("venv")
6950
reticulate::py_config()
51+
# Set reticulate_python env var
7052
writeLines(
7153
paste0("RETICULATE_PYTHON=", reticulate::virtualenv_python("venv")),
7254
Sys.getenv("GITHUB_ENV"))
55+
# Paste string to describe cache
7356
writeLines(
7457
paste0("PY_CACHE_STR=python",
7558
paste(
7659
readLines(".python-version"),
7760
reticulate::virtualenv_python("venv"),
78-
digest::digest(readLines("requirements.txt"))
61+
digest::digest(readLines("requirements.txt")),
62+
sep = "-"
7963
)),
8064
Sys.getenv("GITHUB_ENV"))
8165
@@ -89,7 +73,7 @@ jobs:
8973
id: cache-venv
9074
with:
9175
path: .virtualenvs/r-reticulate
92-
key: setup-venv-${{ runner.os }}-${{ env.PY_CACHE_STR }}
76+
key: setup-venv-${{ env.PY_CACHE_STR }}
9377

9478
- uses: quarto-dev/quarto-actions/setup@v2
9579
with:

0 commit comments

Comments
 (0)