Skip to content

Commit

Permalink
Remove redundant linting dep; Add cache check
Browse files Browse the repository at this point in the history
  • Loading branch information
delatrie committed Apr 18, 2024
1 parent d857349 commit da94632
Showing 1 changed file with 16 additions and 5 deletions.
21 changes: 16 additions & 5 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ jobs:
lint:
name: Static check
runs-on: ubuntu-latest
needs: [commons, pytest-changes, other-changes]
needs: [pytest-changes, other-changes]
if: ${{ needs.pytest-changes.outputs.changed == 'true' || needs.other-changes.outputs.packages != '[]' }}
steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -128,6 +128,11 @@ jobs:
with:
path: dist/
key: commons-${{ github.sha }}
fail-on-cache-miss: true

- name: Check restored cache
run: |
ls dist/allure-python-commons*.tar.gz
- name: Install packages
run: |
Expand Down Expand Up @@ -169,12 +174,18 @@ jobs:
with:
path: dist/
key: commons-${{ github.sha }}
fail-on-cache-miss: true

- name: Check restored cache
run: |
ls dist/allure-python-commons*.tar.gz
- name: Install packages
run: pip install dist/allure-python-commons*.tar.gz &&
pip install ./${{ matrix.package }} &&
pip install -r ./requirements/testing.txt &&
pip install -r ./requirements/testing/${{ matrix.package }}.txt
run: |
pip install dist/allure-python-commons*.tar.gz \
./${{ matrix.package }} \
-r ./requirements/testing.txt \
-r ./requirements/testing/${{ matrix.package }}.txt
- name: Test ${{ matrix.package }}
working-directory: ${{ matrix.package }}
Expand Down

0 comments on commit da94632

Please sign in to comment.