Skip to content

Commit aa49946

Browse files
authored
Merge pull request #3286 from The-OpenROAD-Project-staging/feature/use-venv-in-workflows
Use venv for python dependencies in github actions
2 parents 7553390 + af25042 commit aa49946

File tree

5 files changed

+17
-5
lines changed

5 files changed

+17
-5
lines changed

.github/workflows/github-actions-lint-tcl.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,12 @@ jobs:
1717

1818
- name: Install Dependencies
1919
run: |
20-
python3 -m pip install -U --user tclint==0.4.2
20+
python3 -m venv venv
21+
venv/bin/pip install tclint==0.4.2
2122
2223
- name: Lint
2324
run: |
25+
source venv/bin/activate
2426
tclfmt --version
2527
tclfmt --in-place .
2628
git diff --exit-code

.github/workflows/github-actions-manual-update-rules.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,14 @@ jobs:
2222
python-version: "3.10"
2323
- name: Install Python Packages
2424
run: |
25-
pip install firebase-admin
25+
python3 -m venv venv
26+
venv/bin/pip install firebase-admin
2627
- name: Execute Python Script Update
2728
env:
2829
CREDS_FILE: ${{ secrets.CREDS_FILE }}
2930
API_BASE_URL: ${{ secrets.API_BASE_URL }}
3031
run: |
32+
source venv/bin/activate
3133
if [[ "${{ github.event.inputs.type }}" == "overwrite" ]]; then
3234
python flow/util/updateRules.py --keyFile "${CREDS_FILE}" --apiURL ${API_BASE_URL} --commitSHA $(git rev-parse HEAD) --overwrite
3335
else

.github/workflows/github-actions-update-rules.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,14 @@ jobs:
2424
python-version: "3.10"
2525
- name: Install Python Packages
2626
run: |
27-
pip install firebase-admin
27+
python3 -m venv venv
28+
venv/bin/pip install firebase-admin
2829
- name: Execute Python Script Update
2930
env:
3031
CREDS_FILE: ${{ secrets.CREDS_FILE }}
3132
API_BASE_URL: ${{ secrets.API_BASE_URL }}
3233
run: |
34+
source ./venv/bin/activate
3335
echo ${{ github.event_name }}
3436
echo ${{ github.event.client_payload.type }}
3537
if [[ "${{ github.event_name }}" == "repository_dispatch" && "${{ github.event.client_payload.type }}" == "overwrite" ]]; then

.github/workflows/github-actions-yaml-test.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,13 @@ jobs:
2323
- name: Check if FlowVariables.md is up to date
2424
run: |
2525
git diff --exit-code docs/user/FlowVariables.md
26+
- name: Install dependencies
27+
run: |
28+
python3 -m venv venv
29+
venv/bin/pip install --quiet yamlfix==1.17.0
2630
- name: Run yamlfix check
2731
run: |
28-
pip install --quiet yamlfix==1.17.0
32+
source venv/bin/activate
2933
yamlfix --version
3034
set -x
3135
yamlfix -c yamlfix.toml flow/scripts/variables.yaml

.gitignore

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ flow/rc_model.bin
2929
flow/*.tif.gz
3030
flow/*.def.v
3131

32-
3332
# RePlAce deps
3433
*PORT9.dat
3534
*POST9.dat
@@ -103,3 +102,6 @@ bazel-bin
103102
bazel-out
104103
bazel-OpenROAD-flow-scripts
105104
bazel-testlogs
105+
106+
# python venv
107+
venv/

0 commit comments

Comments
 (0)