Formatting fixes (#161)

* Bump black from 22.12.0 to 24.3.0 (#139)

* Bump black from 22.12.0 to 24.3.0

Bumps [black](https://github.com/psf/black) from 22.12.0 to 24.3.0.
- [Release notes](https://github.com/psf/black/releases)
- [Changelog](https://github.com/psf/black/blob/main/CHANGES.md)
- [Commits](psf/black@22.12.0...24.3.0)

---
updated-dependencies:
- dependency-name: black
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>

* Fix datetime issue

* Hotfix receipt generation

* Update wording

* Try to fix pytest by upgrading google auth

* Don't ask for off-session setup on anonymous donors (#147)

* Bump braces from 3.0.2 to 3.0.3 (#144)

Bumps [braces](https://github.com/micromatch/braces) from 3.0.2 to 3.0.3.
- [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md)
- [Commits](micromatch/braces@3.0.2...3.0.3)

---
updated-dependencies:
- dependency-name: braces
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Leo <[email protected]>

* --- (#143)

updated-dependencies:
- dependency-name: requests
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Leo Kuboschek <[email protected]>

* build(deps): bump pug from 3.0.2 to 3.0.3 (#151)

Bumps [pug](https://github.com/pugjs/pug) from 3.0.2 to 3.0.3.
- [Release notes](https://github.com/pugjs/pug/releases)
- [Commits](https://github.com/pugjs/pug/compare/[email protected]@3.0.3)

---
updated-dependencies:
- dependency-name: pug
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump tqdm from 4.65.0 to 4.66.3 (#142)

Bumps [tqdm](https://github.com/tqdm/tqdm) from 4.65.0 to 4.66.3.
- [Release notes](https://github.com/tqdm/tqdm/releases)
- [Commits](tqdm/tqdm@v4.65.0...v4.66.3)

---
updated-dependencies:
- dependency-name: tqdm
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Leo Kuboschek <[email protected]>
Co-authored-by: Leo <[email protected]>

* Group dependabot updates monthly

* Reformat

* Bump tqdm from 4.65.0 to 4.66.3 (#142)

Bumps [tqdm](https://github.com/tqdm/tqdm) from 4.65.0 to 4.66.3.
- [Release notes](https://github.com/tqdm/tqdm/releases)
- [Commits](tqdm/tqdm@v4.65.0...v4.66.3)

---
updated-dependencies:
- dependency-name: tqdm
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Leo Kuboschek <[email protected]>
Co-authored-by: Leo <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Leo Kuboschek <[email protected]>
Co-authored-by: Leo <[email protected]>

* Fix formatting

* Fix Dockerfile warnings

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Dockerfile

@@ -1,11 +1,11 @@
-FROM alpine/git:v2.26.2-amd64 as version
+FROM alpine/git:v2.26.2-amd64 AS version
 
 ADD .git .git
 RUN git describe --always > /PORTAL_VERSION
 RUN echo "Saved version file containing '$(cat /PORTAL_VERSION)'"
 
 # image for building node dependencies
-FROM node:16-alpine as frontend
+FROM node:16-alpine AS frontend
 
 RUN apk add --no-cache \
     git
@@ -68,40 +68,40 @@ COPY --from=frontend /app/webpack-stats.json /app/webpack-stats.json
 COPY --from=frontend /app/assets/bundles /app/assets/bundles/
 
 # default settings are in MemberManagement
-ENV DJANGO_SETTINGS_MODULE "MemberManagement.docker_settings"
+ENV DJANGO_SETTINGS_MODULE="MemberManagement.docker_settings"
 
 ### ALL THE CONFIGURATION
 
 # disable / enable the devel warning shown on the page
-ENV DJANGO_ENABLE_DEVEL_WARNING "1"
+ENV DJANGO_ENABLE_DEVEL_WARNING="1"
 
 # The secret key used for django
-ENV DJANGO_SECRET_KEY ""
+ENV DJANGO_SECRET_KEY=""
 
 # A comma-seperated list of allowed hosts
-ENV DJANGO_ALLOWED_HOSTS "localhost"
-ENV DJANGO_CSRF_ORIGINS "http://localhost:8000"
+ENV DJANGO_ALLOWED_HOSTS="localhost"
+ENV DJANGO_CSRF_ORIGINS="http://localhost:8000"
 
 # Database settings
 ## Use SQLITE out of the box
-ENV DJANGO_DB_ENGINE "django.db.backends.sqlite3"
-ENV DJANGO_DB_NAME "/data/MemberManagment.db"
-ENV DJANGO_DB_USER ""
-ENV DJANGO_DB_PASSWORD ""
-ENV DJANG_DB_HOST ""
-ENV DJANGO_DB_PORT ""
+ENV DJANGO_DB_ENGINE="django.db.backends.sqlite3"
+ENV DJANGO_DB_NAME="/data/MemberManagment.db"
+ENV DJANGO_DB_USER=""
+ENV DJANGO_DB_PASSWORD=""
+ENV DJANG_DB_HOST=""
+ENV DJANGO_DB_PORT=""
 
 # Stripe keys -- required
-ENV STRIPE_SECRET_KEY ""
-ENV STRIPE_PUBLISHABLE_KEY ""
+ENV STRIPE_SECRET_KEY=""
+ENV STRIPE_PUBLISHABLE_KEY=""
 
-ENV FINALIZE_AUTOMATICALLY "1"
+ENV FINALIZE_AUTOMATICALLY="1"
 
 # GSuite Auth file should be in the data volume
-ENV GSUITE_AUTH_FILE /data/credentials.json
+ENV GSUITE_AUTH_FILE=/data/credentials.json
 
 # Raven -- optional
-ENV DJANGO_RAVEN_DSN ""
+ENV DJANGO_RAVEN_DSN=""
 
 # Collect all the static files at build time
 RUN DJANGO_SECRET_KEY=setup python manage.py collectstatic --noinput
@@ -113,4 +113,4 @@ EXPOSE 80
 ENTRYPOINT ["/app/docker/entrypoint.sh"]
 CMD ["uvicorn", "MemberManagement.asgi:application", "--host", "0.0.0.0", "--port", "80"]
 
-COPY --from=version /PORTAL_VERSION /app/PORTAL_VERSION
+COPY --from=version /PORTAL_VERSION /app/PORTAL_VERSION

MemberManagement/docker_settings.py

@@ -2,6 +2,7 @@
 Django Docker settings for MemberManagement project.
 Reads all relevant setting from the environment
 """
+
 from .settings import *
 
 # No Debugging

alumni/admin/actions.py

@@ -143,7 +143,7 @@ def makeHeaderCell(field):
 
         # Write each of the rows
         copy = queryset.all()
-        for (raw, row) in zip(copy, queryset.values_list(*field_names)):
+        for raw, row in zip(copy, queryset.values_list(*field_names)):
 
             def makeCell(prop):
                 try:

alumni/fields/tier.py

@@ -24,7 +24,7 @@ class TierField(CustomTextChoiceField):
 
     @staticmethod
     def get_description(value):
-        for (k, v) in TierField.CHOICES:
+        for k, v in TierField.CHOICES:
             if k == value:
                 return v
 

alumni/tests/test_002_admin_export.py

@@ -562,7 +562,7 @@ def test_export(self) -> None:
             len(EXPECT_EXPORT),
             "check that the workbook contains the sheet number of columns",
         )
-        for (i, (got, expected)) in enumerate(zip(values, EXPECT_EXPORT)):
+        for i, (got, expected) in enumerate(zip(values, EXPECT_EXPORT)):
             self.assertListEqual(
                 got, expected, "check that column {} is correct".format(i)
             )

alumni/utils.py

@@ -6,8 +6,7 @@
 class ParsingCallback(Protocol):
     """A callback for parsing"""
 
-    def __call__(self, *args: str) -> Any:
-        ...
+    def __call__(self, *args: str) -> Any: ...
 
 
 class CSVParser(object):
@@ -133,7 +132,7 @@ def parse(
 
         # check that the values are of the correct length
         count = len(fields)
-        for (i, v) in enumerate(values):
+        for i, v in enumerate(values):
             if len(v) != count:
                 raise Exception(
                     "Malformed values: Index {} is not of expected length".format(i)
@@ -145,11 +144,11 @@ def parse(
         # prepare a list of results
         results: List[Dict[str, Any]] = [{} for _ in values]
 
-        for (target, idxs) in indexes.items():
+        for target, idxs in indexes.items():
             mapper = mappers[target]
 
             # apply the mapper to each target and save it!
-            for (index, value) in enumerate(values):
+            for index, value in enumerate(values):
                 params: Iterable[str] = map(lambda i: value[i], idxs)
                 results[index][target] = mapper(*params)
 

donation_receipts/views.py

@@ -18,9 +18,9 @@ def download_receipt(request, receipt_id):
     )
 
     response = FileResponse(receipt.receipt_pdf)
-    response[
-        "Content-Disposition"
-    ] = f"filename=Zuwendungsbestätigung vom {receipt.received_on}.pdf"
+    response["Content-Disposition"] = (
+        f"filename=Zuwendungsbestätigung vom {receipt.received_on}.pdf"
+    )
 
     return response
 

payments/tests/test_040_upgrade_noop.py

@@ -25,9 +25,9 @@ def setUp(self) -> None:
         super().setUp()
 
         self.start_tier: str = self.user.alumni.membership.tier
-        self.start_subscription: Optional[
-            str
-        ] = self.user.alumni.subscription.subscription
+        self.start_subscription: Optional[str] = (
+            self.user.alumni.subscription.subscription
+        )
         self.start_time: datetime = self.user.alumni.subscription.start
         self.end_time: Optional[datetime] = self.user.alumni.subscription.end