Upgrade Python dependencies

* Upgrade pre-commit dependencies
* Remove pyvcr
* Add Makefile for compiling Python requirements
* Rename codecov configuration file
* Add flake8 configuration file
* Update CI workflow
* Replace TmpDir helper with pytest fixtures

.flake8

@@ -0,0 +1,16 @@
+[flake8]
+exclude = .tox, .git, __pycache__, .cache, build, dist, *.pyc, *.egg-info, .eggs
+# Error codes:
+# - https://flake8.pycqa.org/en/latest/user/error-codes.html
+# - https://pycodestyle.pycqa.org/en/latest/intro.html#error-codes
+# - https://github.com/PyCQA/flake8-bugbear#list-of-warnings
+#
+# E203: whitespace before `,`, `;` or `:`
+# E402: module level import not at top of file
+# E501: line too long
+# W503: line break before binary operator
+ignore =
+    E203,
+    E402,
+    E501,
+    W503

.github/workflows/test.yml

@@ -6,10 +6,11 @@ on:
     branches:
       - "master"
 jobs:
-  tox:
+  test:
     name: "Test Python ${{ matrix.python-version }}"
     runs-on: "ubuntu-22.04"
     strategy:
+      fail-fast: false
       matrix:
         python-version: [
           "3.8",
@@ -53,6 +54,10 @@ jobs:
         uses: "actions/setup-python@v4"
         with:
           python-version: "3.12"
+          cache: "pip"
+          cache-dependency-path: |
+            requirements.txt
+            requirements-dev.txt
       - name: "Install tox"
         run: |
           python -m pip install --upgrade pip

.pre-commit-config.yaml

@@ -1,16 +1,16 @@
 repos:
 - repo: https://github.com/asottile/pyupgrade
-  rev: v3.10.1
+  rev: v3.15.0
   hooks:
   - id: pyupgrade
     args: [--py38-plus]
 - repo: https://github.com/asottile/reorder_python_imports
-  rev: v3.10.0
+  rev: v3.12.0
   hooks:
   - id: reorder-python-imports
     args: [--py38-plus]
 - repo: https://github.com/psf/black
-  rev: "23.7.0"
+  rev: "23.11.0"
   hooks:
   - id: black
     args: [--safe, --quiet]

Makefile

@@ -0,0 +1,14 @@
+.DEFAULT_GOAL := help
+
+.PHONY: pip-compile pip-upgrade
+
+pip-compile:  ## Compile pip requirements
+	pip-compile --allow-unsafe --output-file=requirements.txt requirements.in
+	pip-compile --allow-unsafe --output-file=requirements-dev.txt requirements-dev.in
+
+pip-upgrade:  ## Upgrade pip requirements
+	pip-compile --allow-unsafe --upgrade --output-file=requirements.txt requirements.in
+	pip-compile --allow-unsafe --upgrade --output-file=requirements-dev.txt requirements-dev.in
+
+help:  ## Print this help message
+	@grep -E '^[a-zA-Z_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-30s\033[0m %s\n", $$1, $$2}'

aips/create_dip.py

@@ -150,6 +150,7 @@ def extract_aip(aip_file, aip_uuid, tmp_dir):
         pass
 
     # Find extracted entry. Assuming it contains the AIP UUID
+    extracted_entry = ""
     for entry in os.listdir(tmp_dir):
         if aip_uuid in entry:
             extracted_entry = os.path.join(tmp_dir, entry)