Add multiple python versions to CI (#25)

.github/workflows/ci.yml

@@ -6,12 +6,15 @@ jobs:
   run_tests:
     name: Style checks and test
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.7", "3.8", "3.9", "3.10", "3.11"]
 
     steps:
       - name: Checkout
         uses: actions/checkout@v3
-      - name: Install dependencies
-        run: make deps
+      - name: Install dependencies - Python ${{ matrix.python-version }}
+        run: PYTHON_VERSION=${{ matrix.python-version }} make deps
       - name: Lint
         run: make check
       - name: Test

Makefile

@@ -11,6 +11,7 @@ DEPS := $(VENV)/.deps
 PROJECT := streetview
 PROJECT_DIR = $(PWD)/$(PROJECT)
 TEST_DIR = $(PWD)/tests
+PYTHON_VERSION ?= 3.11
 PYTHON_CMD := PYTHONPATH=$(PWD) $(VENV)/bin/python
 COVERAGE := 90  # Required code coverage
 
@@ -31,6 +32,7 @@ $(DEPS): environment.yml $(MAMBA)
 	echo "Installing dependencies..."
 	rm -rf $(VENV)
 	$(MAMBA) create --quiet --yes -p $(VENV)
+	$(MAMBA) install --quiet --yes --log-level 4 -p $(VENV) python=$(PYTHON_VERSION) -c conda-forge
 	$(MAMBA) install --quiet --yes --log-level 4 -p $(VENV) -f environment.yml
 	cp environment.yml $(DEPS)
 

environment.yml

@@ -6,19 +6,18 @@ channels:
   - defaults
 
 dependencies:
-  - python=3.11.3
   - requests
   - pillow
   - pydantic
 
   # Testing dependencies
-  - pytest=7.3.1
-  - pytest-cov==4.0.0
-  - pytest-vcr==1.0.2
-  - isort==5.12.0
-  - black==23.3.0
-  - flake8==6.0.0
-  - mypy==1.2.0
+  - pytest
+  - pytest-cov
+  - pytest-vcr
+  - isort
+  - black
+  - flake8
+  - mypy
   - types-requests
   - types-Pillow
   - build

streetview/api.py

@@ -1,4 +1,5 @@
 from io import BytesIO
+from typing import Dict, Union
 
 import requests
 from PIL import Image
@@ -62,7 +63,7 @@ def get_streetview(
     """
 
     url = "https://maps.googleapis.com/maps/api/streetview"
-    params: dict[str, str | int] = {
+    params: Dict[str, Union[str, int]] = {
         "size": "%dx%d" % (width, height),
         "fov": fov,
         "pitch": pitch,

streetview/search.py

@@ -1,5 +1,6 @@
 import json
 import re
+from typing import List, Optional
 
 import requests
 from pydantic import BaseModel
@@ -13,7 +14,7 @@ class Panorama(BaseModel):
     heading: float
     pitch: float
     roll: float
-    date: str | None
+    date: Optional[str]
 
 
 def make_search_url(lat: float, lon: float) -> str:
@@ -41,7 +42,7 @@ def search_request(lat: float, lon: float) -> Response:
     return requests.get(url)
 
 
-def extract_panoramas(text: str) -> list[Panorama]:
+def extract_panoramas(text: str) -> List[Panorama]:
     """
     Given a valid response from the panoids endpoint, return a list of all the
     panoids.
@@ -86,7 +87,7 @@ def extract_panoramas(text: str) -> list[Panorama]:
     ]
 
 
-def search_panoramas(lat: float, lon: float) -> list[Panorama]:
+def search_panoramas(lat: float, lon: float) -> List[Panorama]:
     """
     Gets the closest panoramas (ids) to the GPS coordinates.
     """