Skip to content

Commit

Permalink
chore: clean up
Browse files Browse the repository at this point in the history
  • Loading branch information
agateau-gg committed May 5, 2023
1 parent 7c62aad commit 2575101
Showing 1 changed file with 14 additions and 19 deletions.
33 changes: 14 additions & 19 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,24 +80,13 @@ jobs:
python -m pip install --upgrade pip
python -m pip install --upgrade pipenv==2022.10.4
pipenv install --system --dev
pip show urllib3
python -c "import urllib3 ; print(urllib3.__version__) ; print(urllib3.__file__)"
# Hack: workaround urllib3 still being installed on windows-2022 builder
pip install --force-reinstall 'urllib3<2'
pip show urllib3
python -c "import urllib3 ; print(urllib3.__version__) ; print(urllib3.__file__)"
python -c "import vcr"
- name: Install Windows dev dependencies
if: matrix.os == 'windows-2022'
run: |
# Those are win32-only dependencies from pytest
python -m pip install atomicwrites colorama
pip freeze | grep urllib
python -c "import vcr"
- name: Override base Docker image used for functional tests on Windows
if: matrix.os == 'windows-2022'
Expand All @@ -109,20 +98,26 @@ jobs:
- name: Ensure a clean package installation
run: |
python -c 'import sys ; print(sys.executable)'
pip install wheel check-wheel-contents
pip freeze | grep urllib
python setup.py clean --all bdist_wheel
# The created wheel (.whl) file will be found and analyzed within the `dist/` folder
check-wheel-contents dist/
pip freeze | grep urllib
python -c "import vcr"
- name: Run unit tests
run: |
pip freeze | grep urllib
pytest --disable-pytest-warnings --disable-socket tests/unit
python -c "import vcr"
coverage run --source ggshield -m pytest --disable-pytest-warnings --disable-socket tests/unit
- name: Gather coverage report
run: |
coverage report --fail-under=80
coverage xml
- uses: codecov/codecov-action@v1
with:
file: ./coverage.xml
flags: unittests
name: codecov-umbrella
fail_ci_if_error: false

- name: Run functional tests
# See note about steps requiring the GITGUARDIAN_API at the top of this file
Expand Down

0 comments on commit 2575101

Please sign in to comment.