Fix linter config parsing errors (#1010)

* switch to pyproject.toml

* run tests on master as well this will fix coverage reporting issues

.flake8

@@ -0,0 +1,13 @@
+[flake8]
+max-line-length = 88
+extend-ignore = E203,E501,E231
+exclude =
+  .git,
+  .gitignore,
+  *.pot,
+  *.py[co],
+  __pycache__,
+  .venv,
+  */migrations/*,
+  */static/CACHE/*,
+  docs

.github/workflows/deployment.yaml

@@ -26,7 +26,7 @@ env:
   CONTAINER_NAME_CRON: 'care-celery-beat'
 jobs:
   test:
-    if: github.ref == 'refs/heads/production'
+    if: github.ref == 'refs/heads/production' || github.ref == 'refs/heads/master'
     runs-on: ubuntu-latest
     env:
       DATABASE_URL: postgres://postgres:postgres@localhost:5432/care

.github/workflows/linter.yml

@@ -25,6 +25,6 @@ jobs:
           VALIDATE_PYTHON_FLAKE8: true
           VALIDATE_PYTHON_ISORT: true
           LINTER_RULES_PATH: /
-          PYTHON_BLACK_CONFIG_FILE: "setup.cfg"
-          PYTHON_FLAKE8_CONFIG_FILE: "setup.cfg"
-          PYTHON_ISORT_CONFIG_FILE: "setup.cfg"
+          # PYTHON_BLACK_CONFIG_FILE: "pyproject.toml"
+          PYTHON_FLAKE8_CONFIG_FILE: ".flake8"
+          PYTHON_ISORT_CONFIG_FILE: "pyproject.toml"

.pre-commit-config.yaml

@@ -1,6 +1,5 @@
 exclude: "docs|node_modules|migrations|.git|.tox"
 default_stages: [commit]
-fail_fast: true
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
@@ -16,18 +15,19 @@ repos:
     rev: 5.10.1
     hooks:
       - id: isort
-        args: ["--profile", "black"]
+        additional_dependencies: ["isort[pyproject]"]
 
   - repo: https://github.com/psf/black
     rev: 22.3.0
     hooks:
       - id: black
+        args: ["--config=pyproject.toml"]
 
   - repo: https://github.com/PyCQA/flake8
     rev: 4.0.1
     hooks:
       - id: flake8
-        args: ["--config=setup.cfg"]
+        args: ["--config=pyproject.toml"]
         additional_dependencies: [flake8-isort]
 
 ci:

docker/docker-entrypoint.sh

@@ -6,7 +6,7 @@ python manage.py migrate
 
 echo "All migrations have been made successfully"
 
-if ["${DJANGO_DEBUG,,}" == "true"]; then
+if [[ "${DJANGO_DEBUG,,}" == "true" ]]; then
   python -m debugpy --wait-for-client --listen 0.0.0.0:9876 manage.py runserver_plus 0.0.0.0:9000
 else
   python manage.py runserver 0.0.0.0:9000

pyproject.toml

@@ -0,0 +1,62 @@
+[tool.coverage.run]
+branch = true
+source = ["care", "config"]
+parallel = true
+concurrency = ["multiprocessing"]
+
+
+[tool.coverage.report]
+omit = [
+  "*/tests/*",
+  "*/migrations/*",
+  "*/asgi.py",
+  "*/wsgi.py",
+  "manage.py",
+  ".venv/*"
+]
+exclude_lines = [
+  "pragma: no cover",
+  "raise NotImplementedError"
+]
+ignore_errors = true
+
+
+[tool.isort]
+profile = "black"
+known_third_party = [
+  "allauth",
+  "boto3",
+  "celery",
+  "crispy_forms",
+  "dateparser",
+  "dateutil",
+  "django",
+  "django_filters",
+  "django_rest_passwordreset",
+  "djangoql",
+  "djqscsv",
+  "drf_extra_fields",
+  "drf_yasg",
+  "dry_rest_permissions",
+  "environ",
+  "fernet_fields",
+  "freezegun",
+  "hardcopy",
+  "healthy_django",
+  "jsonschema",
+  "jwt",
+  "location_field",
+  "multiselectfield",
+  "partial_index",
+  "phonenumber_field",
+  "phonenumbers",
+  "pytz",
+  "pywebpush",
+  "ratelimit",
+  "requests",
+  "rest_framework",
+  "rest_framework_nested",
+  "rest_framework_simplejwt",
+  "sentry_sdk",
+  "simple_history"
+]

requirements/local.txt

@@ -18,7 +18,7 @@ django-extensions==2.2.8  # https://github.com/django-extensions/django-extensio
 # -------------------------------
 factory-boy==2.12.0  # https://github.com/FactoryBoy/factory_boy
 freezegun==0.3.15  # https://github.com/spulec/freezegun
-coverage==6.4.1  # https://github.com/nedbat/coveragepy
+coverage[toml]==6.4.1  # https://github.com/nedbat/coveragepy
 django-coverage-plugin==2.0.3
 tblib==1.7.0  # https://github.com/ionelmc/python-tblib
 # WatchDog