Merge pull request #65 from pinecone-io/packaging-improvements

Packaging improvements: multi-python testing and looser depedency version constraints

Author: benjaminran

.github/workflows/pr.yaml

@@ -6,15 +6,24 @@ jobs:
   run-tests:
     name: Run tests
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: [3.6, 3.7, 3.8, 3.9]
 
     steps:
-    - uses: actions/checkout@v2
-    - uses: actions/setup-python@v2
+    - uses: actions/checkout@v1
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v2
       with:
-        python-version: 3.7
+        python-version: ${{ matrix.python-version }}
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install tox tox-gh-actions
 
     - name: Run tests
-      run: make tests
+      run: make tests TOX_PARALLEL_NO_SPINNER=1
 
     - name: Build Python client
       run: make package

Makefile

@@ -11,7 +11,7 @@ develop:
 
 tests:
 	# skipping flake8 for now
-	pip3 install --upgrade --quiet tox && tox -e py3
+	pip3 install --upgrade --quiet tox && TOX_SKIP_ENV='flake|docs' tox
 
 docs:
 	echo skipping temporarily...

requirements.txt

@@ -1,9 +1,9 @@
-requests==2.25.1
-pyyaml==5.4
-loguru==0.5.3
-typing-extensions==3.10.0.0
-sentry-sdk==1.1.0
-dnspython==2.1.0
+requests>=2.19.0
+pyyaml>=5.4
+loguru>=0.5.0
+typing-extensions>=3.7.4
+sentry-sdk>=1.0.0
+dnspython>=2.0.0
 # openapi generated client:
 python_dateutil >= 2.5.3
-urllib3 >= 1.25.3
+urllib3 >= 1.21.1

tox.ini

@@ -1,12 +1,21 @@
 [tox]
-envlist = py3,flake8,docs
+envlist = py{36,37,38,39}-pcgrpc_{yes,no}-old_deps_{yes,no},flake8,docs
+skip_missing_interpreters = true
 
 [testenv]
 setenv = HOSTNAME=hostname
 deps=
+  old_deps_yes: requests==2.19.0
+  old_deps_yes: pyyaml==5.4
+  old_deps_yes: loguru==0.5.0
+  old_deps_yes: typing-extensions==3.7.4
+  old_deps_yes: sentry-sdk==1.0.0
+  old_deps_yes: dnspython==2.0.0
+  old_deps_yes: python_dateutil==2.5.3
+  old_deps_yes: urllib3==1.21.1
   -r {toxinidir}/requirements.txt
-  -r {toxinidir}/requirements-grpc.txt
   -r {toxinidir}/test-requirements.txt
+  pcgrpc_yes: -r {toxinidir}/requirements-grpc.txt
 commands =
   pytest --cov=pinecone --timeout=120 tests/unit {posargs}