Merge pull request #6 from reddit/dynamic_config

Decider

Author: wuyiqicc

.dockerignore

@@ -8,7 +8,5 @@ __pycache__/
 /.dockerignore
 /.git
 /Dockerfile
-/tests/range_variant_tests/data/original_zk_config.json
-/tests/range_variant_tests/data/range_variant_zk_config.json
-/tests/range_variant_tests/data/output.json
+/tests/range_variant_tests/data/*
 venv*

.github/workflows/python-package.yaml

@@ -11,8 +11,9 @@ jobs:
 
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
-        python-version: [3.7, 3.8, 3.9]
+        python-version: ['3.7', '3.8', '3.9', '3.10']
 
     steps:
     - uses: actions/checkout@v2
@@ -35,7 +36,7 @@ jobs:
 
     - name: Lint
       run: |
-        make lint
+        make lint PYTHON_VERSION=${{ matrix.python-version }}
 
     - name: Test
       run: |

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.8
+FROM python:3.9
 
 WORKDIR /src
 COPY requirements*.txt .

Makefile

@@ -1,7 +1,7 @@
 SOURCE_ROOT := reddit_experiments/
-PYTHON_SOURCE = $(shell find $(SOURCE_ROOT) tests/ setup.py -name '*.py')
+PYTHON_SOURCE = $($(shell) find $(SOURCE_ROOT) tests/ setup.py -name '*.py')
 REORDER_PYTHON_IMPORTS := reorder-python-imports --py3-plus --separate-from-import --separate-relative
-
+PYTHON_VERSION ?= 3.9
 
 .PHONY: fmt
 fmt:
@@ -14,7 +14,7 @@ lint:
 	$(REORDER_PYTHON_IMPORTS) --diff-only $(PYTHON_SOURCE)
 	black --diff --check $(PYTHON_SOURCE)
 	flake8 $(SOURCE_ROOT)
-	mypy $(SOURCE_ROOT)
+	mypy $(SOURCE_ROOT) --python-version $(PYTHON_VERSION)
 
 .PHONY: test
 test: