Skip to content

Commit

Permalink
Run workflows through pdm
Browse files Browse the repository at this point in the history
  • Loading branch information
mogres committed Oct 17, 2024
1 parent ecccf39 commit a848020
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/analyze.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,14 @@ jobs:
- uses: actions/checkout@v4
- uses: ./.github/actions/dependencies
- name: Pack test recipe
shell: bash
run: |
pack -r cellpack/tests/recipes/v2/test_spheres.json -c cellpack/tests/packing-configs/test_config.json
pdm run pack -r cellpack/tests/recipes/v2/test_spheres.json -c cellpack/tests/packing-configs/test_config.json
- name: Modify JSON with PR sub_directory path
run: |
jq --arg branch_name "${{ github.ref_name }}" '.create_report.output_image_location |= "https://cellpack-results.s3.us-west-2.amazonaws.com/\($branch_name)/spheresSST/figures"' cellpack/tests/analysis-configs/PR_analysis_config.json > cellpack/tests/analysis-configs/PR_analysis_config_temp.json
mv cellpack/tests/analysis-configs/PR_analysis_config_temp.json cellpack/tests/analysis-configs/PR_analysis_config.json
- name: Run analysis code
run: analyze -r cellpack/tests/recipes/v2/test_spheres.json -a cellpack/tests/analysis-configs/PR_analysis_config.json -p cellpack/tests/outputs/test_spheres/spheresSST
run: pdm run analyze -r cellpack/tests/recipes/v2/test_spheres.json -a cellpack/tests/analysis-configs/PR_analysis_config.json -p cellpack/tests/outputs/test_spheres/spheresSST
- name: Upload results
uses: actions/upload-artifact@v4
with:
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
- uses: ./.github/actions/dependencies
- name: Test with pytest
run: |
pytest --cov cellpack/tests/
pdm run pytest --cov cellpack/tests/
- name: Upload codecov
uses: codecov/codecov-action@v4

Expand All @@ -46,10 +46,10 @@ jobs:
- uses: ./.github/actions/dependencies
- name: Lint with flake8
run: |
flake8 cellpack --count --verbose --show-source --statistics --ignore=E501,E277,W503,E203
pdm run flake8 cellpack --count --verbose --show-source --statistics --ignore=E501,E277,W503,E203
- name: Check with black
run: |
black --check cellpack
pdm run black --check cellpack
publish:
if: success() && startsWith(github.ref, 'refs/tags/')
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/cleanup-firebase.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,4 @@ jobs:
FIREBASE_TOKEN: ${{ secrets.FIREBASE_TOKEN }}
FIREBASE_EMAIL: ${{ secrets.FIREBASE_EMAIL }}
run: |
python cellpack/bin/cleanup_tasks.py
pdm run python cellpack/bin/cleanup_tasks.py

0 comments on commit a848020

Please sign in to comment.