fixup! [WIP] pre-commit and ruff

Author: swrichards

.github/ISSUE_TEMPLATE/feature_request.md

@@ -19,4 +19,4 @@ A description of the idea or need, possibly supplemented with a specific solutio
 The added value for business operations or services
 
 ### Additional context
-Any other context or screenshots about the feature request
+Any other context or screenshots about the feature request

.github/workflows/pre-commit.yml

@@ -9,21 +9,21 @@ jobs:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 0
-      
+
       - name: Set up Python
         uses: actions/setup-python@v5
         with:
           python-version: '3.x'
-      
+
       - name: Install pre-commit
         run: pip install pre-commit
-      
+
       - name: Get changed files
         id: changed-files
         run: |
           echo "files=$(git diff --name-only origin/main...HEAD | tr '\n' ' ')" >> $GITHUB_OUTPUT
-      
+
       - name: Run pre-commit
         continue-on-error: true
         run: |
-          pre-commit run --files ${{ steps.changed-files.outputs.files }}
+          pre-commit run --files ${{ steps.changed-files.outputs.files }}

.gitignore

@@ -49,4 +49,3 @@ src/open_inwoner/static/webfonts/fa-solid-900.woff2
 src/open_inwoner/static/webfonts/fa-v4compatibility.ttf
 src/open_inwoner/static/webfonts/fa-v4compatibility.woff2
 src/open_inwoner/static/fonts/material-icons/
-

.pre-commit-config.yaml

@@ -1,10 +1,19 @@
 repos:
-- repo: https://github.com/astral-sh/ruff-pre-commit
-  # Ruff version.
-  rev: v0.8.4
-  hooks:
-    # Run the linter.
-    - id: ruff
-      args: [ --fix ]
-    # Run the formatter.
-    - id: ruff-format
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v5.0.0
+    hooks:
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+      - id: mixed-line-ending
+      - id: check-merge-conflict
+      - id: check-toml
+      - id: debug-statements
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    # Ruff version.
+    rev: v0.8.4
+    hooks:
+      # Run the linter.
+      - id: ruff
+        args: [--fix]
+      # Run the formatter.
+      - id: ruff-format

.python-version

@@ -1 +1 @@
-3.11
+3.11

AUTHORS

@@ -4,7 +4,7 @@ Maykin Media team was happy to participate in such a socially significant projec
 
 Here is an inevitably incomplete list of MUCH-APPRECIATED CONTRIBUTORS --
 people who laid the groundwork, have submitted patches, reported bugs, added
-translations, helped answer questions, and generally made the components that 
+translations, helped answer questions, and generally made the components that
 much better:
 
     Alex de Landgraaf <[email protected]>
@@ -21,4 +21,4 @@ A big THANK YOU goes to:
     ideas: the municipalities of Deventer, Enschede, Groningen, Leeuwarden,
     Zaanstad and Zwolle.
 
-    All people behind Common Ground for their enthusiasm and vision.
+    All people behind Common Ground for their enthusiasm and vision.

CHANGELOG.rst

@@ -4,4 +4,3 @@ Change history
 
 
 See the Github releases: https://github.com/maykinmedia/open-inwoner/releases
-

CODE_OF_CONDUCT.md

@@ -70,4 +70,4 @@ members of the project's leadership.
 This Code of Conduct is adapted from the [Contributor Covenant][homepage], version 1.4,
 available at https://www.contributor-covenant.org/version/1/4/code-of-conduct.html
 
-[homepage]: https://www.contributor-covenant.org
+[homepage]: https://www.contributor-covenant.org

CONTRIBUTING.md

@@ -72,4 +72,4 @@ to a release branch.
 [issues]: https://github.com/maykinmedia/open-inwoner/issues
 [new_issue]: https://github.com/maykinmedia/open-inwoner/issues/new/choose
 [Standard_for_Public_Code]: https://standard.publiccode.net
-[repository]: https://github.com/maykinmedia/open-inwoner
+[repository]: https://github.com/maykinmedia/open-inwoner

LICENSE.md

@@ -215,15 +215,15 @@ Deze openbare licentie van de Europese Unie („EUPL”) is van toepassing op he
 of op een andere wijze zijn bereidheid te kennen heeft gegeven krachtens de EUPL in licentie te geven.
 
 1.Definities
-In deze licentie wordt verstaan onder:  
-— „de licentie”:de onderhavige licentie;  
-— „het oorspronkelijke werk”:het werk dat of de software die door de licentiegever krachtens deze licentie wordt verspreid of medegedeeld, en dat/die beschikbaar is als broncode en, in voorkomend geval, ook als uitvoerbare code;  
-— „bewerkingen”:de werken of software die de licentiehouder kan creëren op grond van het oorspronkelijke werk of wijzigingen ervan. In deze licentie wordt niet gedefinieerd welke mate van wijziging of afhankelijkheid van het oorspronkelijke werk vereist is om een werk als een bewerking te kunnen aanmerken; dat wordt bepaald conform het auteursrecht dat van toepassing is in de in artikel 15 bedoelde staat;  
-— „het werk”:het oorspronkelijke werk of de bewerkingen ervan;  
-— „de broncode”:de voor mensen leesbare vorm van het werk, die het gemakkelijkste door mensen kan worden bestudeerd en gewijzigd;  
-— „de uitvoerbare code”:elke code die over het algemeen is gecompileerd en is bedoeld om door een computer als een programma te worden uitgevoerd;  
-— „de licentiegever”:de natuurlijke of rechtspersoon die het werk krachtens de licentie verspreidt of mededeelt;  
-— „bewerker(s)”:elke natuurlijke of rechtspersoon die het werk krachtens de licentie wijzigt of op een andere wijze bijdraagt tot de totstandkoming van een bewerking;  
+In deze licentie wordt verstaan onder:
+— „de licentie”:de onderhavige licentie;
+— „het oorspronkelijke werk”:het werk dat of de software die door de licentiegever krachtens deze licentie wordt verspreid of medegedeeld, en dat/die beschikbaar is als broncode en, in voorkomend geval, ook als uitvoerbare code;
+— „bewerkingen”:de werken of software die de licentiehouder kan creëren op grond van het oorspronkelijke werk of wijzigingen ervan. In deze licentie wordt niet gedefinieerd welke mate van wijziging of afhankelijkheid van het oorspronkelijke werk vereist is om een werk als een bewerking te kunnen aanmerken; dat wordt bepaald conform het auteursrecht dat van toepassing is in de in artikel 15 bedoelde staat;
+— „het werk”:het oorspronkelijke werk of de bewerkingen ervan;
+— „de broncode”:de voor mensen leesbare vorm van het werk, die het gemakkelijkste door mensen kan worden bestudeerd en gewijzigd;
+— „de uitvoerbare code”:elke code die over het algemeen is gecompileerd en is bedoeld om door een computer als een programma te worden uitgevoerd;
+— „de licentiegever”:de natuurlijke of rechtspersoon die het werk krachtens de licentie verspreidt of mededeelt;
+— „bewerker(s)”:elke natuurlijke of rechtspersoon die het werk krachtens de licentie wijzigt of op een andere wijze bijdraagt tot de totstandkoming van een bewerking;
 — „de licentiehouder” of „u”:elke natuurlijke of rechtspersoon die het werk onder de voorwaarden van de licentie gebruikt;  — „verspreiding” of „mededeling”:het verkopen, geven, uitlenen, verhuren, verspreiden, mededelen, doorgeven, of op een andere wijze online of offline beschikbaar stellen van kopieën van het werk of het verlenen van toegang tot de essentiële functies ervan ten behoeve van andere natuurlijke of rechtspersonen.
 
 2.Draagwijdte van de uit hoofde van de licentie verleende rechten
@@ -263,7 +263,7 @@ De oorspronkelijke licentiegever garandeert dat hij houder is van het hierbij ve
 Het werk is een werk in ontwikkeling, dat voortdurend door vele bewerkers wordt verbeterd. Het is een onvoltooid werk, dat bijgevolg nog tekortkomingen of programmeerfouten („bugs”) kan vertonen, die onlosmakelijk verbonden zijn met dit soort ontwikkeling. Om die reden wordt het werk op grond van de licentie verstrekt „zoals het is” en zonder enige garantie met betrekking tot het werk te geven, met inbegrip van, maar niet beperkt tot garanties met betrekking tot de verhandelbaarheid, de geschiktheid voor een specifiek doel, de afwezigheid van tekortkomingen of fouten, de nauwkeurigheid, de eerbiediging van andere intellectuele-eigendomsrechten dan het in artikel 6 van deze licentie bedoelde auteursrecht. Deze uitsluiting van garantie is een essentieel onderdeel van de licentie en een voorwaarde voor de verlening van rechten op het werk.
 
 8.Uitsluiting van aansprakelijkheid
-Behoudens in het geval van een opzettelijke fout of directe schade aan natuurlijke personen, is de licentiegever in geen enkel geval aansprakelijk voor ongeacht welke directe of indirecte, materiële of immateriële schade die voortvloeit uit de licentie of het gebruik van het werk, met inbegrip van, maar niet beperkt tot schade als gevolg van het verlies van goodwill, verloren werkuren, een computerdefect of computerfout, het verlies van gegevens, of enige andere commerciële schade, zelfs indien de licentiegever werd gewezen op de mogelijkheid van dergelijke schade. De licentiegever is echter aansprakelijk op grond van de wetgeving inzake productaansprakelijkheid, voor zover deze wetgeving op het werk van toepassing is. 
+Behoudens in het geval van een opzettelijke fout of directe schade aan natuurlijke personen, is de licentiegever in geen enkel geval aansprakelijk voor ongeacht welke directe of indirecte, materiële of immateriële schade die voortvloeit uit de licentie of het gebruik van het werk, met inbegrip van, maar niet beperkt tot schade als gevolg van het verlies van goodwill, verloren werkuren, een computerdefect of computerfout, het verlies van gegevens, of enige andere commerciële schade, zelfs indien de licentiegever werd gewezen op de mogelijkheid van dergelijke schade. De licentiegever is echter aansprakelijk op grond van de wetgeving inzake productaansprakelijkheid, voor zover deze wetgeving op het werk van toepassing is.
 
 9.Aanvullende overeenkomsten
 Bij de verspreiding van het werk kunt u ervoor kiezen een aanvullende overeenkomst te sluiten, waarin de verplichtingen of diensten overeenkomstig deze licentie worden omschreven. Indien deze verplichtingen worden aanvaard, kunt u echter alleen in eigen naam en onder eigen verantwoordelijkheid handelen, en dus niet in naam van de oorspronkelijke licentiegever of een bewerker, en kunt u voorts alleen handelen indien u ermee instemt alle bewerkers schadeloos te stellen, te verdedigen of te vrijwaren met betrekking tot de aansprakelijkheid van of vorderingen tegen deze bewerkers op grond van het feit dat u een garantie of aanvullende aansprakelijkheid hebt aanvaard.

SECURITY.rst

@@ -3,61 +3,61 @@
 Security policy
 ===============
 
-The development team is strongly committed to responsible reporting and 
-disclosure of security-related issues. As such, we’ve adopted and follow a set 
-of policies which conform to that ideal and are geared toward allowing us to 
+The development team is strongly committed to responsible reporting and
+disclosure of security-related issues. As such, we’ve adopted and follow a set
+of policies which conform to that ideal and are geared toward allowing us to
 deliver timely security updates to the official distribution of Open Inwoner.
 
 Reporting security issues
 -------------------------
 
-**Short version: please report security issues by emailing 
+**Short version: please report security issues by emailing
 [email protected].**
 
-If you discover security issues in Open Inwoner or related projects under the 
-same organization, we request you to disclose these in a *responsible* way by 
+If you discover security issues in Open Inwoner or related projects under the
+same organization, we request you to disclose these in a *responsible* way by
 mailing to [email protected].
 
-It is extremely useful if you have a reproducible test case and/or clear steps 
+It is extremely useful if you have a reproducible test case and/or clear steps
 on how to reproduce the vulnerability.
 
-Please do not report security issues on the public Github issue tracker, as 
-this makes it visible which exploits exist before a fix is available, 
+Please do not report security issues on the public Github issue tracker, as
+this makes it visible which exploits exist before a fix is available,
 potentially comprising a lot of unprotected instances.
 
-Once you’ve submitted an issue via email, you should receive an acknowledgment 
-from a member of the security team as soon as possible, and depending on the 
+Once you’ve submitted an issue via email, you should receive an acknowledgment
+from a member of the security team as soon as possible, and depending on the
 action to be taken, you may receive further followup emails.
 
 Timeline of the process
 -----------------------
 
-Open Inwoner community support is provided by `Maykin Media`_. The community 
+Open Inwoner community support is provided by `Maykin Media`_. The community
 support team is responsible for the handling of security issues.
 
-1. The recipients of the report first validate if there is indeed a (possible) 
+1. The recipients of the report first validate if there is indeed a (possible)
    issue.
 
 2. After validation, we confirm that we received the report and if it is indeed
    a valid issue.
 
-3. We have a private Github repository accessible only to the community support 
-   team. In this repository, an issue is created for the vulnerability where 
+3. We have a private Github repository accessible only to the community support
+   team. In this repository, an issue is created for the vulnerability where
    the impact and possible solutions are discussed.
 
-4. The next step is to create a (draft) Github security advisory, which is only 
-   visible to the repository administrators and community support team. 
+4. The next step is to create a (draft) Github security advisory, which is only
+   visible to the repository administrators and community support team.
    Severity and impact will be established here.
 
 5. If appropriate, we request a `CVE identifier`_ from Github.
 
 6. A patch is implemented, reviewed and tested in a private fork.
 
-7. When the fix is tested and release coordination is done, the fix is merged 
-   into the primary repository. The security advisory and release are 
+7. When the fix is tested and release coordination is done, the fix is merged
+   into the primary repository. The security advisory and release are
    published. All managed instances should be updated.
 
-8. The release and security vulnerability are communicated to the community. 
+8. The release and security vulnerability are communicated to the community.
    This includes an announcement on `commonground.nl`_.
 
 

bin/generate_cms_fixture.sh

@@ -11,5 +11,3 @@
 # Note: you will likely need to restart uwsgi/supervisor/docker afterwards for apphooks to be registered properly
 
 src/manage.py dumpdata --indent=4 --natural-primary cms.treenode cms.placeholder cms.page cms.title cms.cmsplugin extensions.commonextension > src/open_inwoner/conf/fixtures/cms-page.json
-
-

docker-nginx-default.conf

@@ -11,4 +11,4 @@ server {
         client_max_body_size 100M;
         proxy_pass   http://web:8000;
     }
-}
+}

docker/openklant/fixtures/db.json

@@ -215,4 +215,4 @@
 			"object_type": "zaak"
 		}
 	}
-]
+]