.pre-commit-config.yaml

@@ -1,36 +1,31 @@
 # Pre-commit hooks to run tests and ensure code is cleaned.
 # See https://pre-commit.com for more information
 repos:
-- repo: https://github.com/asottile/pyupgrade
+-   repo: local
-  rev: v3.15.0
+    hooks:
-  hooks:
+    -   id: pylint
-  - id: pyupgrade
+        name: pylint
-    args: ['--keep-percent-format', '--py37-plus']
+        entry: pylint
-- repo: https://github.com/psf/black
+        language: system
-  rev: 23.11.0
+        types: [python]
-  hooks:
+        require_serial: true
-  - id: black
+-   repo: https://github.com/PyCQA/flake8
-    args: ['--target-version', 'py37', '--line-length', '100']
+    rev: 6.0.0
-- repo: https://github.com/PyCQA/isort
+    hooks:
-  rev: 5.12.0
+    -   id: flake8
-  hooks:
+        args: ['--max-line-length=100']
-  - id: isort
+-   repo: https://github.com/asottile/pyupgrade
-    args: ['--profile', 'black', '--line-length', '100']
+    rev: v3.3.1
-- repo: https://github.com/PyCQA/flake8
+    hooks:
-  rev: 6.1.0
+    - id: pyupgrade
-  hooks:
+      args: ['--keep-percent-format', '--py37-plus']
-  - id: flake8
+-   repo: https://github.com/psf/black
-    args: ['--max-line-length=100']
+    rev: 22.12.0
-- repo: local
+    hooks:
-  hooks:
+    - id: black
-  - id: pylint
+      args: ['--target-version', 'py37', '--line-length', '100']
-    name: pylint
+-   repo: https://github.com/PyCQA/isort
-    entry: pylint
+    rev: 5.12.0
-    language: system
+    hooks:
-    types: [python]
+    - id: isort
-    require_serial: true
+      args: ['--profile', 'black', '--line-length', '100']
-- repo: https://github.com/PyCQA/bandit
-  rev: 1.7.5
-  hooks:
-  - id: bandit
-    args: [--skip, "B101", --recursive, "mylib"]

check_forgejo_upgrade

@@ -21,7 +21,7 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 import argparse
 import logging
 import re
-import subprocess  # nosec
+import subprocess
 import sys
 
 import requests
@@ -59,7 +59,7 @@ logging.debug("Command use to retreive current version of Forgejo: %s", " ".join
 OUTPUT = None
 EXCEPTION = None
 try:
-    OUTPUT = subprocess.check_output(cmd)  # nosec
+    OUTPUT = subprocess.check_output(cmd)
     logging.debug("Output:\n%s", OUTPUT)
     m = re.search("version ([^ ]+) built", OUTPUT.decode("utf8", errors="ignore"))
     if m:
@@ -81,32 +81,19 @@ CURRENT = CURRENT.replace("+", "-")
 logging.debug("Cleaned current version: %s", CURRENT)
 
 LATEST = None
-LATEST_INT = None
 try:
     logging.debug("Get releases RSS feed from %s...", options.url)
     r = requests.get(options.url, timeout=options.timeout)
     logging.debug("Data retreive:\n%s", r.text)
     data = xmltodict.parse(r.text)
-    versions = {}
     for item in data["rss"]["channel"]["item"]:
         version = re.sub("^v", "", item["title"])
         if not options.include_rc and "-rc" in version:
             logging.debug("Ignore release candidate %s", version)
             continue
-        version_int = int(re.sub(r"[\.-]", "000", version))
+        LATEST = version
-        logging.debug("Found version %s (%s)", version, version_int)
+        break
-        if not LATEST_INT or LATEST_INT < version_int:
+except Exception:  # pylint: disable=broad-except
-            if LATEST:
-                logging.debug(
-                    "Version %s considered as newer than %s, override latest version",
-                    version,
-                    LATEST,
-                )
-            LATEST = version
-            LATEST_INT = version_int
-        else:
-            logging.debug("Version %s considered as oldest than %s", version, LATEST)
-except Exception:  # pylint: disable=broad-except  # nosec
     pass
 logging.debug("Latest version: %s", LATEST)