Skip to content

Commit

Permalink
Build: Move pip installs to Python venv
Browse files Browse the repository at this point in the history
Required due to brew adopting PEP 668
https://github.com/orgs/Homebrew/discussions/3404

Currently only required on macOS, and attempting to change breaks build
on Linux and Windows, so only changing macOS for now.
  • Loading branch information
mikebeaton committed May 1, 2024
1 parent 92cac23 commit 864e934
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/workflows/build_uncrustify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@ jobs:

- name: Install Dependencies
run: |
python3 -m venv ~/pyenv
source ~/pyenv/bin/activate
python3 -m pip install -r requirements.txt
- name: CI Bootstrap
Expand All @@ -29,6 +31,7 @@ jobs:
- name: Build Uncrustify
run: |
source ~/pyenv/bin/activate
cd uncstrap && python3 ./uncstrap.py -b
echo "UNC_SHA=$(cat unc-sha.txt)" >> $GITHUB_ENV
Expand Down
6 changes: 5 additions & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,14 @@ jobs:

- name: Install Dependencies
run: |
python3 -m venv ~/pyenv
source ~/pyenv/bin/activate
python3 -m pip install -r requirements.txt
- name: Run prospector
run: python3 -m prospector . -P ./prospector/profile.yml > prospector_result.txt || exit 1
run: |
source ~/pyenv/bin/activate
python3 -m prospector . -P ./prospector/profile.yml > prospector_result.txt || exit 1
- name: Upload prospector result to Artifacts
uses: actions/upload-artifact@v3
Expand Down

0 comments on commit 864e934

Please sign in to comment.