Skip to content

Commit

Permalink
Use msbuild setup action
Browse files Browse the repository at this point in the history
  • Loading branch information
codebytere committed May 15, 2020
1 parent 4c3d33d commit 3cedd7e
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 13 deletions.
9 changes: 4 additions & 5 deletions .github/workflows/test-python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@ jobs:
uses: actions/setup-python@v1
with:
python-version: ${{ matrix.python-version }}
- name: Add msbuild to PATH
if: matrix.os == 'windows-latest'
uses: microsoft/setup-msbuild@v1.0.0
- name: Install dependencies
run: |
python -m pip install --upgrade pip
Expand All @@ -40,10 +43,6 @@ jobs:
- name: Test with pytest
shell: bash
run: |
if [ "$RUNNER_OS" == "Windows" ]; then
GYP_MSVS_VERSION=2015 GYP_MSVS_OVERRIDE_PATH="C:\\Dummy" python -m pytest
else
python -m pytest
fi
python -m pytest
# - name: Run doctests with pytest
# run: pytest --doctest-modules
13 changes: 5 additions & 8 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,22 +30,19 @@ jobs:
python-version: ${{ matrix.python }}
env:
PYTHON_VERSION: ${{ matrix.python }}
- name: Add msbuild to PATH
if: matrix.os == 'windows-latest'
uses: microsoft/setup-msbuild@v1.0.0
- name: Install Dependencies
run: |
python -m pip install --upgrade pip flake8 pytest==4.6.6
python -m pip install --upgrade pip flake8 pytest
npm install
- name: Lint Python
run: |
python -m flake8 --version
python -m flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
python -m flake8 . --count --exit-zero --ignore=E111,E114,W503 --max-complexity=10 --max-line-length=127 --statistics
- name: Run Tests
shell: bash
run: |
echo "Testing with Python $(python --version 2>&1)"
npm test
if [ "$RUNNER_OS" == "Windows" ]; then
GYP_MSVS_VERSION=2015 GYP_MSVS_OVERRIDE_PATH="C:\\Dummy" python -m pytest
else
python -m pytest
fi
python -m pytest

0 comments on commit 3cedd7e

Please sign in to comment.