Skip to content

Commit

Permalink
Cleaned up some of the lints
Browse files Browse the repository at this point in the history
  • Loading branch information
VisLab committed Sep 26, 2024
1 parent 194b230 commit f36ecc5
Show file tree
Hide file tree
Showing 46 changed files with 5,954 additions and 5,868 deletions.
24 changes: 15 additions & 9 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,28 +56,34 @@ jobs:
pip install -r requirements.txt
pip install -r docs/requirements.txt
# Run flake8 only for Python 3.9
- name: Lint with flake8
if: matrix.python-version == '3.9'
run: |
# stop the build if there are Python syntax errors or undefined names
pip install flake8
flake8 . --count --show-source --statistics --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
# exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
- name: Test with unittest
env:
HED_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: |
coverage run -m unittest
if [ "${{ matrix.python-version }}" == "3.9" ]; then
pip install coverage
coverage run -m unittest discover tests
else
python -m unittest tests
fi
- name: Run spec_test coverage
env:
HED_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: coverage run --append -m unittest spec_tests/test_errors.py
continue-on-error: true
run: |
if ["${{ matrix.python-version }}" == "3.9" ]; then
coverage run --append -m unittest discover tests/spec_tests
ls -a
continue-on-error: true
if
- name: List files in workspace
run: ls -a

- name: Archive code coverage results
if: ${{matrix.python-version == '3.9'}}
uses: actions/upload-artifact@v4
Expand Down
405 changes: 205 additions & 200 deletions tests/errors/test_error_reporter.py

Large diffs are not rendered by default.

Loading

0 comments on commit f36ecc5

Please sign in to comment.