Skip to content

Commit

Permalink
TST: Remove -OO testrun from CI (#901)
Browse files Browse the repository at this point in the history
We needed it because we were manipulating a docstring programmatically.

As long as we don't rely on assert / docstrings being present, we don't need to test with -OO.
  • Loading branch information
MartinThoma authored May 26, 2022
1 parent 787c784 commit 68515c9
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions .github/workflows/github-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ["3.6", "3.7", "3.10.1", "3.8", "3.9", "3.10"]
python-version: ["3.6", "3.7", "3.8", "3.9", "3.10"]

steps:
- name: Checkout Code
Expand All @@ -47,11 +47,6 @@ jobs:
- name: Test with pytest
run: |
python -m coverage run --parallel-mode -m pytest tests -vv
if: matrix.python-version != '3.10.1'
- name: Test with pytest (OO flag)
run: |
python -OO -m coverage run --parallel-mode -m pytest tests -vv
if: matrix.python-version == '3.10.1'
- name: Test with mypy
run : |
mypy PyPDF2 --show-error-codes --disallow-untyped-defs --disallow-incomplete-defs
Expand Down

0 comments on commit 68515c9

Please sign in to comment.