Skip to content

Commit 3845f7d

Browse files
committed
More fixes
- Dropped Python 2.7 testing, not sure why this was on the repo but we probably don't want that. Set python_requires to 3.7 and made CI test 3.7 and 3.9 for now. - Converted to pytest for its simplicity - Made tests write output files to a temporary path to keep things from getting cluttered. - Cleaned up some of the imports - Added a fix to install typing_compat and typing_extensions when using Python 3.7.
1 parent bdc2e04 commit 3845f7d

File tree

6 files changed

+196
-268
lines changed

6 files changed

+196
-268
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
strategy:
1414
fail-fast: false
1515
matrix:
16-
python-version: [ 2.7, 3.7 , 3.9 ]
16+
python-version: [ 3.7 , 3.9 ]
1717

1818
steps:
1919
- uses: actions/checkout@v2

setup.cfg

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,10 @@ install_requires =
3939
attrs
4040
cattrs
4141
docstring-parser
42+
typing_extensions;python_version<'3.8'
43+
typing_compat;python_version<'3.8'
4244

43-
python_requires = >=3.6
45+
python_requires = >=3.7
4446
include_package_data = True
4547
package_dir =
4648
=src

test_all.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,4 @@ python document.py
88

99
cd ..
1010

11-
cd modelspec/test
12-
pytest -v
11+
pytest tests

tests/netid.json

Lines changed: 0 additions & 39 deletions
This file was deleted.

tests/netid.yaml

Lines changed: 0 additions & 26 deletions
This file was deleted.

0 commit comments

Comments
 (0)