pre-commit hook updates (#25)

Co-authored-by: Fabian Braun <[email protected]>

Author: bencipher

.github/PULL_REQUEST_TEMPLATE.md

@@ -26,5 +26,5 @@ Use 'x' to check each item: [x] I have ...
 * [ ] I have opened this pull request against ``master``
 * [ ] I have added or modified the tests when changing logic
 * [ ] I have followed [the conventional commits guidelines](https://www.conventionalcommits.org/) to add meaningful information into the changelog
-* [ ] I have read the [contribution guidelines ](https://github.com/django-cms/django-cms/blob/develop/CONTRIBUTING.rst) and I have joined #workgroup-pr-review on 
+* [ ] I have read the [contribution guidelines ](https://github.com/django-cms/django-cms/blob/develop/CONTRIBUTING.rst) and I have joined #workgroup-pr-review on
 [Slack](https://www.django-cms.org/slack) to find a “pr review buddy” who is going to review my pull request.

.pre-commit-config.yaml

@@ -0,0 +1,47 @@
+ci:
+    autofix_commit_msg: |
+        ci: auto fixes from pre-commit hooks
+        for more information, see https://pre-commit.ci
+    autofix_prs: false
+    autoupdate_commit_msg: 'ci: pre-commit autoupdate'
+    autoupdate_schedule: monthly
+
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.1.0
+    hooks:
+    -   id: trailing-whitespace
+    -   id: end-of-file-fixer
+    -   id: check-yaml
+    -   id: check-added-large-files
+    -   id: check-merge-conflict
+    -   id: debug-statements
+    -   id: mixed-line-ending
+    -   id: trailing-whitespace
+-   repo: https://github.com/asottile/pyupgrade
+    rev: v3.2.0
+    hooks:
+      - id: pyupgrade
+        args: ["--py39-plus"]
+-   repo: https://github.com/adamchainz/django-upgrade
+    rev: '1.4.0'
+    hooks:
+      - id: django-upgrade
+        args: [--target-version, "4.2"]
+-   repo: https://github.com/PyCQA/flake8
+    rev: 4.0.1
+    hooks:
+      - id: flake8
+-   repo: https://github.com/asottile/yesqa
+    rev: v1.3.0
+    hooks:
+      - id: yesqa
+-   repo: https://github.com/pycqa/isort
+    rev: 5.10.1
+    hooks:
+      - id: isort
+-   repo: https://github.com/codespell-project/codespell
+    rev: v2.1.0
+    hooks:
+      - id: codespell
+        args: ["--ignore-words-list", "ist, oder, alle"]

LICENSE

@@ -21,4 +21,4 @@ DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
 LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
 ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
-SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

djangocms_transfer/cms_plugins.py

@@ -33,10 +33,6 @@ def get_plugin_urls(self):
         ]
         return urlpatterns
 
-    def get_extra_placeholder_menu_items(self, request, placeholder):  # noqa
-        # django-cms 3.4 compatibility
-        return self.get_extra_placeholder_menu_items(request, placeholder)
-
     def get_extra_global_plugin_menu_items(self, request, plugin):
         # django-cms 3.4 compatibility
         return self.get_extra_plugin_menu_items(request, plugin)
@@ -73,7 +69,7 @@ def get_extra_plugin_menu_items(cls, request, plugin):
             ),
         ]
 
-    @classmethod  # noqa
+    @classmethod
     def get_extra_placeholder_menu_items(cls, request, placeholder):  # noqa
         data = urlencode(
             {
@@ -188,8 +184,8 @@ def export_plugins_view(cls, request):
 
         # TODO: Check permissions
         filename = form.get_filename()
-        response = HttpResponse(form.run_export(), content_type="application/json")
-        response["Content-Disposition"] = "attachment; filename={}".format(filename)
+        response = HttpResponse(form.run_export(), content_type='application/json')
+        response['Content-Disposition'] = f'attachment; filename={filename}'
         return response
 
 

djangocms_transfer/locale/de/LC_MESSAGES/django.po

@@ -2,10 +2,10 @@
 # Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
 # This file is distributed under the same license as the PACKAGE package.
 # FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
-# 
+#
 # Translators:
 # Angelo Dini <[email protected]>, 2019
-# 
+#
 #, fuzzy
 msgid ""
 msgstr ""

djangocms_transfer/locale/es/LC_MESSAGES/django.po

@@ -2,7 +2,7 @@
 # Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
 # This file is distributed under the same license as the PACKAGE package.
 # FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
-# 
+#
 #, fuzzy
 msgid ""
 msgstr ""

djangocms_transfer/locale/fr/LC_MESSAGES/django.po

@@ -2,7 +2,7 @@
 # Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
 # This file is distributed under the same license as the PACKAGE package.
 # FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
-# 
+#
 #, fuzzy
 msgid ""
 msgstr ""

tests/test_migrations.py

@@ -25,5 +25,5 @@ def test_for_missing_migrations(self):
             # the "no changes" exit code is 0
             status_code = "0"
 
-        if status_code == "1":
-            self.fail("There are missing migrations:\n {}".format(output.getvalue()))
+        if status_code == '1':
+            self.fail(f'There are missing migrations:\n {output.getvalue()}')