feat(make_readme): add some tests

2024-03-08 05:10:47 +01:00
27 changed files with 436 additions and 768 deletions
--- a/README.md
+++ b/README.md
@ -1,6 +1,6 @@
 # YunoHost application catalog

-<img alt="YunoHost logo" src="https://avatars.githubusercontent.com/u/1519495?s=200&v=4" width=80><img alt="Package logo" src="https://yunohost.org/user/images/yunohost_package.png" width=80>
+<img src="https://avatars.githubusercontent.com/u/1519495?s=200&v=4" width=80><img src="https://yunohost.org/user/images/yunohost_package.png" width=80>

 This repository contains the default YunoHost app catalog, as well as related
 tools that can be run manually or automatically.
@ -16,7 +16,7 @@ them such as their category or maintenance state. This file is regularly read by

 - You can browse [the contributor documentation](https://yunohost.org/contributordoc)
 - If you are not familiar with Git/GitHub, you can have a look at our [homemade guide](https://yunohost.org/packaging_apps_git)
- Don't hesitate to reach for help on the dedicated [application packaging chatroom](https://yunohost.org/chat_rooms)... we can even schedule an audio meeting to help you get started!
+- Don't hesitate to reach for help on the dedicated [application packaging chatroom](https://yunohost.org/chat_rooms) ... we can even schedule an audio meeting to help you get started!

 ## How to add your app to the application catalog

@ -46,9 +46,9 @@ App example addition:

 ```toml
 [your_app]
-antifeatures = [ "deprecated-software" ]   # Replace with the appropriate category id found in antifeatures.toml, remove if no relevant antifeature applies
+antifeatures = [ "deprecated-software" ]   # Remove if no relevant antifeature applies
 potential_alternative_to = [ "YouTube" ]   # Indicate if your app can be thought of as an alternative to popular proprietary services (or remove if none applies)
-category = "foobar"                        # Replace with the appropriate category id found in categories.toml, don't invent a category
+category = "foobar"                        # Replace with the appropriate category id found in categories.toml
 state = "working"
 url = "https://github.com/YunoHost-Apps/your_app_ynh"
 ```
--- a/apps.toml
+++ b/apps.toml
@ -502,13 +502,6 @@ state = "working"
 subtags = [ "wiki" ]
 url = "https://github.com/YunoHost-Apps/cowyo_ynh"

-[crabfit]
-category = "productivity_and_management"
-potential_alternative_to = [ "Doodle", "OpenSondage" ]
-state = "working"
-subtags = [ "poll" ]
-url = "https://github.com/YunoHost-Apps/crabfit_ynh"
-
 [cryptpad]
 category = "office"
 level = 8
--- a/logos/crabfit.png
+++ b/logos/crabfit.png
--- a/store/app.py
+++ b/store/app.py
@ -149,14 +149,7 @@ def star_app(app_id, action):
    if app_id not in get_catalog()["apps"] and app_id not in get_wishlist():
        return _("App %(app_id) not found", app_id=app_id), 404
    if not session.get("user", {}):
-        return (
-            _("You must be logged in to be able to star an app")
-            + "<br/><br/>"
-            + _(
-                "Note that, due to various abuses, we restricted login on the app store to 'trust level 1' users.<br/><br/>'Trust level 1' is obtained after interacting a minimum with the forum, and more specifically: entering at least 5 topics, reading at least 30 posts, and spending at least 10 minutes reading posts."
-            ),
-            401,
-        )
+        return _("You must be logged in to be able to star an app") + "<br/><br/>" + _("Note that, due to various abuses, we restricted login on the app store to 'trust level 1' users.<br/><br/>'Trust level 1' is obtained after interacting a minimum with the forum, and more specifically: entering at least 5 topics, reading at least 30 posts, and spending at least 10 minutes reading posts."), 401

    app_star_folder = os.path.join(".stars", app_id)
    app_star_for_this_user = os.path.join(
@ -199,13 +192,7 @@ def add_to_wishlist():
    if request.method == "POST":
        user = session.get("user", {})
        if not user:
-            errormsg = (
-                _("You must be logged in to submit an app to the wishlist")
-                + "<br/><br/>"
-                + _(
-                    "Note that, due to various abuses, we restricted login on the app store to 'trust level 1' users.<br/><br/>'Trust level 1' is obtained after interacting a minimum with the forum, and more specifically: entering at least 5 topics, reading at least 30 posts, and spending at least 10 minutes reading posts."
-                )
-            )
+            errormsg = _("You must be logged in to submit an app to the wishlist") + "<br/><br/>" + _("Note that, due to various abuses, we restricted login on the app store to 'trust level 1' users.<br/><br/>'Trust level 1' is obtained after interacting a minimum with the forum, and more specifically: entering at least 5 topics, reading at least 30 posts, and spending at least 10 minutes reading posts.")
            return render_template(
                "wishlist_add.html",
                locale=get_locale(),
@ -233,33 +220,12 @@ def add_to_wishlist():
        website = request.form["website"].strip().replace("\n", "")
        license = request.form["license"].strip().replace("\n", "")

-        boring_keywords_to_check_for_people_not_reading_the_instructions = [
-            "free",
-            "open source",
-            "open-source",
-            "self-hosted",
-            "simple",
-            "lightweight",
-            "light-weight",
-            "léger",
-            "best",
-            "most",
-            "fast",
-            "rapide",
-            "flexible",
-            "puissante",
-            "puissant",
-            "powerful",
-            "secure",
-        ]
+        boring_keywords_to_check_for_people_not_reading_the_instructions = ["free", "open source", "open-source", "self-hosted", "simple", "lightweight", "light-weight", "léger", "best", "most", "fast", "rapide", "flexible", "puissante", "puissant", "powerful", "secure"]

        checks = [
            (
-                check_wishlist_submit_ratelimit(session["user"]["username"]) is True
-                and session["user"]["bypass_ratelimit"] is False,
-                _(
-                    "Proposing wishlist additions is limited to once every 15 days per user. Please try again in a few days."
-                ),
+                check_wishlist_submit_ratelimit(session['user']['username']) is True and session['user']['bypass_ratelimit'] is False,
+                _("Proposing wishlist additions is limited to once every 15 days per user. Please try again in a few days.")
            ),
            (len(name) >= 3, _("App name should be at least 3 characters")),
            (len(name) <= 30, _("App name should be less than 30 characters")),
@ -293,22 +259,13 @@ def add_to_wishlist():
                _("App name contains special characters"),
            ),
            (
-                all(
-                    keyword not in description.lower()
-                    for keyword in boring_keywords_to_check_for_people_not_reading_the_instructions
-                ),
-                _(
-                    "Please focus on what the app does, without using marketing, fuzzy terms, or repeating that the app is 'free' and 'self-hostable'."
-                ),
+                all(keyword not in description.lower() for keyword in boring_keywords_to_check_for_people_not_reading_the_instructions),
+                _("Please focus on what the app does, without using marketing, fuzzy terms, or repeating that the app is 'free' and 'self-hostable'.")
            ),
            (
-                description.lower().split()[0] != name
-                and (
-                    len(description.split()) == 1
-                    or description.lower().split()[1] not in ["is", "est"]
-                ),
-                _("No need to repeat the name of the app. Focus on what the app does."),
-            ),
+                description.lower().split()[0] != name and (len(description.split()) == 1 or description.lower().split()[1] not in ["is", "est"]),
+                _("No need to repeat the name of the app. Focus on what the app does.")
+            )
        ]

        for check, errormsg in checks:
@ -343,8 +300,7 @@ def add_to_wishlist():
                successmsg=None,
                errormsg=_(
                    "An entry with the name %(slug)s already exists in the wishlist, instead, you can <a href='%(url)s'>add a star to the app to show your interest</a>.",
-                    slug=slug,
-                    url=url,
+                    slug=slug, url=url,
                ),
            )

@ -368,7 +324,7 @@ def add_to_wishlist():
            url = "https://github.com/YunoHost/apps/pulls?q=is%3Apr+is%3Aopen+wishlist"
            errormsg = _(
                "Failed to create the pull request to add the app to the wishlist… Maybe there's already <a href='%(url)s'>a waiting PR for this app</a>? Else, please report the issue to the YunoHost team.",
-                url=url,
+                url=url
            )
            return render_template(
                "wishlist_add.html",
@ -422,7 +378,7 @@ Description: {description}
            url=url,
        )

-        save_wishlist_submit_for_ratelimit(session["user"]["username"])
+        save_wishlist_submit_for_ratelimit(session['user']['username'])

        return render_template(
            "wishlist_add.html",
@ -489,17 +445,10 @@ def sso_login_callback():

    uri_to_redirect_to_after_login = session.get("uri_to_redirect_to_after_login")

-    if "trust_level_1" not in user_data["groups"][0].split(","):
-        return (
-            _("Unfortunately, login was denied.")
-            + "<br/><br/>"
-            + _(
-                "Note that, due to various abuses, we restricted login on the app store to 'trust level 1' users.<br/><br/>'Trust level 1' is obtained after interacting a minimum with the forum, and more specifically: entering at least 5 topics, reading at least 30 posts, and spending at least 10 minutes reading posts."
-            ),
-            403,
-        )
+    if "trust_level_1" not in user_data['groups'][0].split(','):
+        return _("Unfortunately, login was denied.") + "<br/><br/>" + _("Note that, due to various abuses, we restricted login on the app store to 'trust level 1' users.<br/><br/>'Trust level 1' is obtained after interacting a minimum with the forum, and more specifically: entering at least 5 topics, reading at least 30 posts, and spending at least 10 minutes reading posts."), 403

-    if "staff" in user_data["groups"][0].split(","):
+    if "staff" in user_data['groups'][0].split(','):
        bypass_ratelimit = True
    else:
        bypass_ratelimit = False
--- a/store/gunicorn.py
+++ b/store/gunicorn.py
@ -1,14 +1,13 @@
 import os
-
 install_dir = os.path.dirname(__file__)
-command = f"{install_dir}/venv/bin/gunicorn"
+command = f'{install_dir}/venv/bin/gunicorn'
 pythonpath = install_dir
 workers = 4
-user = "appstore"
-bind = f"unix:{install_dir}/sock"
-pid = "/run/gunicorn/appstore-pid"
-errorlog = "/var/log/appstore/error.log"
-accesslog = "/var/log/appstore/access.log"
+user = 'appstore'
+bind = f'unix:{install_dir}/sock'
+pid = '/run/gunicorn/appstore-pid'
+errorlog = '/var/log/appstore/error.log'
+accesslog = '/var/log/appstore/access.log'
 access_log_format = '%({X-Real-IP}i)s %({X-Forwarded-For}i)s %(h)s %(l)s %(u)s %(t)s "%(r)s" %(s)s %(b)s "%(f)s" "%(a)s"'
-loglevel = "warning"
+loglevel = 'warning'
 capture_output = True
--- a/store/translations/es/LC_MESSAGES/messages.mo
+++ b/store/translations/es/LC_MESSAGES/messages.mo
--- a/store/translations/es/LC_MESSAGES/messages.po
+++ b/store/translations/es/LC_MESSAGES/messages.po
@ -8,21 +8,19 @@ msgstr ""
 "Project-Id-Version: PROJECT VERSION\n"
 "Report-Msgid-Bugs-To: EMAIL@ADDRESS\n"
 "POT-Creation-Date: 2024-03-05 19:36+0100\n"
-"PO-Revision-Date: 2024-03-09 17:32+0000\n"
-"Last-Translator: cri <cri@cri.cl>\n"
-"Language-Team: Spanish <https://translate.yunohost.org/projects/yunohost/"
-"apps/es/>\n"
+"PO-Revision-Date: 2024-02-21 06:05+0100\n"
+"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
+"Language-Team: es <LL@li.org>\n"
 "Language: es\n"
 "MIME-Version: 1.0\n"
 "Content-Type: text/plain; charset=UTF-8\n"
 "Content-Transfer-Encoding: 8bit\n"
-"Plural-Forms: nplurals=2; plural=n != 1;\n"
-"X-Generator: Weblate 5.3.1\n"
+"Plural-Forms: nplurals=2; plural=(n != 1);\n"
 "Generated-By: Babel 2.14.0\n"

 #: app.py:150
 msgid "App %(app_id) not found"
-msgstr "App %(app_id) no encontrada"
+msgstr ""

 #: app.py:152
 msgid "You must be logged in to be able to star an app"
--- a/store/translations/fr/LC_MESSAGES/messages.mo
+++ b/store/translations/fr/LC_MESSAGES/messages.mo
--- a/store/translations/fr/LC_MESSAGES/messages.po
+++ b/store/translations/fr/LC_MESSAGES/messages.po
@ -8,16 +8,15 @@ msgstr ""
 "Project-Id-Version: PROJECT VERSION\n"
 "Report-Msgid-Bugs-To: EMAIL@ADDRESS\n"
 "POT-Creation-Date: 2024-03-05 19:36+0100\n"
-"PO-Revision-Date: 2024-03-09 04:14+0000\n"
+"PO-Revision-Date: 2024-02-27 19:19+0000\n"
 "Last-Translator: OniriCorpe <oniricorpe@disroot.org>\n"
-"Language-Team: French <https://translate.yunohost.org/projects/yunohost/apps/"
-"fr/>\n"
 "Language: fr\n"
+"Language-Team: French "
+"<https://translate.yunohost.org/projects/yunohost/apps/fr/>\n"
+"Plural-Forms: nplurals=2; plural=n > 1;\n"
 "MIME-Version: 1.0\n"
 "Content-Type: text/plain; charset=utf-8\n"
 "Content-Transfer-Encoding: 8bit\n"
-"Plural-Forms: nplurals=2; plural=n > 1;\n"
-"X-Generator: Weblate 5.3.1\n"
 "Generated-By: Babel 2.14.0\n"

 #: app.py:150
@ -255,7 +254,7 @@ msgstr "Documentation officielle pour les admins"

 #: templates/app.html:142
 msgid "Official user documentation"
-msgstr "Documentation officielle d'utilisation"
+msgstr "Documentation officielle pour les utilisateur·ice·s"

 #: templates/app.html:143
 msgid "Official code repository"
@ -395,8 +394,8 @@ msgid ""
 "advise against their installation and advise users to find alternatives."
 msgstr ""
 "Cela signifie que le développeur ne les mettra plus à jour. Nous "
-"décourageons fortement leur installation et vous conseillons de vous tourner "
-"vers des alternatives."
+"décourageons fortement leur installation et conseillons aux utilisateurs "
+"de se tourner vers des alternatives."

 #: templates/index.html:10
 msgid "Application Store"
@ -461,8 +460,8 @@ msgstr "Vous devez être connecté·e pour proposer une app pour la liste de sou
 #: templates/wishlist_add.html:40
 msgid "Due to abuses, only one proposal every 15 days per user is allowed."
 msgstr ""
-"En raison d'abus, la proposition d'app est limitée à une tous les 15 jours "
-"par personne."
+"En raison d'abus, la proposition d'app est limitée à une tous les 15 "
+"jours par utilisateur·ice."

 #: templates/wishlist_add.html:43
 msgid ""
@ -528,3 +527,4 @@ msgstr "Envoyer"

 #~ msgid "Please check the license of the app your are proposing"
 #~ msgstr "Merci de vérifier la licence de l'app que vous proposez"
+
--- a/store/utils.py
+++ b/store/utils.py
@ -93,7 +93,6 @@ def get_stars():
 get_stars.cache_checksum = None
 get_stars()

-
 def check_wishlist_submit_ratelimit(user):

    dir_ = os.path.join(".wishlist_ratelimit")
@ -102,10 +101,7 @@ def check_wishlist_submit_ratelimit(user):

    f = os.path.join(dir_, md5(user.encode()).hexdigest())

-    return not os.path.exists(f) or (time.time() - os.path.getmtime(f)) > (
-        15 * 24 * 3600
-    )  # 15 days
-
+    return not os.path.exists(f) or (time.time() - os.path.getmtime(f)) > (15 * 24 * 3600) # 15 days

 def save_wishlist_submit_for_ratelimit(user):

@ -182,9 +178,9 @@ def get_app_md_and_screenshots(app_folder, infos):
                if entry.is_file() and ext in ("png", "jpg", "jpeg", "webp", "gif"):
                    with open(entry.path, "rb") as img_file:
                        data = base64.b64encode(img_file.read()).decode("utf-8")
-                        infos["screenshot"] = (
-                            f"data:image/{ext};charset=utf-8;base64,{data}"
-                        )
+                        infos[
+                            "screenshot"
+                        ] = f"data:image/{ext};charset=utf-8;base64,{data}"
                    break

    ram_build_requirement = infos["manifest"]["integration"]["ram"]["build"]
--- a/tools/app_caches.py
+++ b/tools/app_caches.py
@ -9,11 +9,8 @@ from typing import Any

 import tqdm

-from appslib.utils import (
-    REPO_APPS_ROOT,  # pylint: disable=import-error
-    get_catalog,
-    git_repo_age,
-)
+from appslib.utils import (REPO_APPS_ROOT,  # pylint: disable=import-error
+                           get_catalog, git_repo_age)
 from git import Repo


@ -34,8 +31,7 @@ def app_cache_clone(app: str, infos: dict[str, str]) -> None:
        infos["url"],
        to_path=app_cache_folder(app),
        depth=git_depths.get(infos["state"], git_depths["default"]),
-        single_branch=True,
-        branch=infos.get("branch", "master"),
+        single_branch=True, branch=infos.get("branch", "master"),
    )


--- a/tools/appslib/utils.py
+++ b/tools/appslib/utils.py
@ -23,14 +23,10 @@ def git(cmd: list[str], cwd: Optional[Path] = None) -> str:
    if cwd:
        full_cmd.extend(["-C", str(cwd)])
    full_cmd.extend(cmd)
-    return (
-        subprocess.check_output(
-            full_cmd,
-            # env=my_env,
-        )
-        .strip()
-        .decode("utf-8")
-    )
+    return subprocess.check_output(
+        full_cmd,
+        # env=my_env,
+    ).strip().decode("utf-8")


 def git_repo_age(path: Path) -> Union[bool, int]:
@ -46,8 +42,7 @@ def get_catalog(working_only: bool = False) -> dict[str, dict[str, Any]]:
    catalog = toml.load((REPO_APPS_ROOT / "apps.toml").open("r", encoding="utf-8"))
    if working_only:
        catalog = {
-            app: infos
-            for app, infos in catalog.items()
+            app: infos for app, infos in catalog.items()
            if infos.get("state") != "notworking"
        }
    return catalog
--- a/tools/autopatches/autopatch.py
+++ b/tools/autopatches/autopatch.py
@ -7,9 +7,7 @@ import sys

 import requests

-catalog = requests.get(
-    "https://raw.githubusercontent.com/YunoHost/apps/master/apps.json"
-).json()
+catalog = requests.get("https://raw.githubusercontent.com/YunoHost/apps/master/apps.json").json()

 my_env = os.environ.copy()
 my_env["GIT_TERMINAL_PROMPT"] = "0"
@ -46,19 +44,15 @@ def git(cmd, in_folder=None):
 def progressbar(it, prefix="", size=60, file=sys.stdout):
    it = list(it)
    count = len(it)
-
    def show(j, name=""):
        name += "          "
-        x = int(size * j / count)
-        file.write(
-            "%s[%s%s] %i/%i %s\r" % (prefix, "#" * x, "." * (size - x), j, count, name)
-        )
+        x = int(size*j/count)
+        file.write("%s[%s%s] %i/%i %s\r" % (prefix, "#"*x, "."*(size-x), j,  count, name))
        file.flush()
-
    show(0)
    for i, item in enumerate(it):
        yield item
-        show(i + 1, item["id"])
+        show(i+1, item["id"])
    file.write("\n")
    file.flush()

@ -69,10 +63,7 @@ def build_cache():
        folder = os.path.join(".apps_cache", app["id"])
        reponame = app["url"].rsplit("/", 1)[-1]
        git(f"clone --quiet --depth 1 --single-branch {app['url']} {folder}")
-        git(
-            f"remote add fork https://{login}:{token}@github.com/{login}/{reponame}",
-            in_folder=folder,
-        )
+        git(f"remote add fork https://{login}:{token}@github.com/{login}/{reponame}", in_folder=folder)


 def apply(patch):
@ -90,11 +81,7 @@ def diff():

    for app in apps():
        folder = os.path.join(".apps_cache", app["id"])
-        if bool(
-            subprocess.check_output(f"cd {folder} && git diff", shell=True)
-            .strip()
-            .decode("utf-8")
-        ):
+        if bool(subprocess.check_output(f"cd {folder} && git diff", shell=True).strip().decode("utf-8")):
            print("\n\n\n")
            print("=================================")
            print("Changes in : " + app["id"])
@ -105,50 +92,35 @@ def diff():

 def push(patch):

-    title = (
-        "[autopatch] "
-        + open(os.path.join("patches", patch, "pr_title.md")).read().strip()
-    )
+    title = "[autopatch] " + open(os.path.join("patches", patch, "pr_title.md")).read().strip()

    def diff_not_empty(app):
        folder = os.path.join(".apps_cache", app["id"])
-        return bool(
-            subprocess.check_output(f"cd {folder} && git diff", shell=True)
-            .strip()
-            .decode("utf-8")
-        )
+        return bool(subprocess.check_output(f"cd {folder} && git diff", shell=True).strip().decode("utf-8"))

    def app_is_on_github(app):
        return "github.com" in app["url"]

-    apps_to_push = [
-        app for app in apps() if diff_not_empty(app) and app_is_on_github(app)
-    ]
+    apps_to_push = [app for app in apps() if diff_not_empty(app) and app_is_on_github(app)]

    with requests.Session() as s:
        s.headers.update({"Authorization": f"token {token}"})
        for app in progressbar(apps_to_push, "Forking: ", 40):
-            app["repo"] = app["url"][len("https://github.com/") :].strip("/")
+            app["repo"] = app["url"][len("https://github.com/"):].strip("/")
            fork_if_needed(app["repo"], s)

        for app in progressbar(apps_to_push, "Pushing: ", 40):
-            app["repo"] = app["url"][len("https://github.com/") :].strip("/")
+            app["repo"] = app["url"][len("https://github.com/"):].strip("/")
            app_repo_name = app["url"].rsplit("/", 1)[-1]
            folder = os.path.join(".apps_cache", app["id"])
            current_branch = git(f"symbolic-ref --short HEAD", in_folder=folder)
            git(f"reset origin/{current_branch}", in_folder=folder)
-            git(
-                ["commit", "-a", "-m", title, "--author='Yunohost-Bot <>'"],
-                in_folder=folder,
-            )
+            git(["commit", "-a", "-m", title, "--author='Yunohost-Bot <>'"], in_folder=folder)
            try:
                git(f"remote remove fork", in_folder=folder)
            except Exception:
                pass
-            git(
-                f"remote add fork https://{login}:{token}@github.com/{login}/{app_repo_name}",
-                in_folder=folder,
-            )
+            git(f"remote add fork https://{login}:{token}@github.com/{login}/{app_repo_name}", in_folder=folder)
            git(f"push fork {current_branch}:{patch} --quiet --force", in_folder=folder)
            create_pull_request(app["repo"], patch, current_branch, s)

@ -169,15 +141,11 @@ def fork_if_needed(repo, s):

 def create_pull_request(repo, patch, base_branch, s):

-    PR = {
-        "title": "[autopatch] "
-        + open(os.path.join("patches", patch, "pr_title.md")).read().strip(),
-        "body": "This is an automatic PR\n\n"
-        + open(os.path.join("patches", patch, "pr_body.md")).read().strip(),
-        "head": login + ":" + patch,
-        "base": base_branch,
-        "maintainer_can_modify": True,
-    }
+    PR = {"title": "[autopatch] " + open(os.path.join("patches", patch, "pr_title.md")).read().strip(),
+          "body": "This is an automatic PR\n\n" + open(os.path.join("patches", patch, "pr_body.md")).read().strip(),
+          "head": login + ":" + patch,
+          "base": base_branch,
+          "maintainer_can_modify": True}

    r = s.post(github_api + f"/repos/{repo}/pulls", json.dumps(PR))

@ -191,8 +159,7 @@ def main():

    action = sys.argv[1]
    if action == "--help":
-        print(
-            """
+        print("""
    Example usage:

 # Init local git clone for all apps
@ -206,8 +173,7 @@ def main():

 # Push and create pull requests on all apps with non-empty diff
 ./autopatch.py --push explicit-php-version-in-deps
-"""
-        )
+""")

    elif action == "--build-cache":
        build_cache()
--- a/tools/autoupdate_app_sources/autoupdate_app_sources.py
+++ b/tools/autoupdate_app_sources/autoupdate_app_sources.py
@ -21,20 +21,10 @@ import github
 # add apps/tools to sys.path
 sys.path.insert(0, str(Path(__file__).parent.parent))

-from rest_api import (
-    GithubAPI,
-    GitlabAPI,
-    GiteaForgejoAPI,
-    RefType,
-)  # noqa: E402,E501 pylint: disable=import-error,wrong-import-position
+from rest_api import GithubAPI, GitlabAPI, GiteaForgejoAPI, RefType  # noqa: E402,E501 pylint: disable=import-error,wrong-import-position
 import appslib.logging_sender  # noqa: E402 pylint: disable=import-error,wrong-import-position
-from appslib.utils import (
-    REPO_APPS_ROOT,
-    get_catalog,
-)  # noqa: E402 pylint: disable=import-error,wrong-import-position
-from app_caches import (
-    app_cache_folder,
-)  # noqa: E402 pylint: disable=import-error,wrong-import-position
+from appslib.utils import REPO_APPS_ROOT, get_catalog  # noqa: E402 pylint: disable=import-error,wrong-import-position
+from app_caches import app_cache_folder  # noqa: E402 pylint: disable=import-error,wrong-import-position


 STRATEGIES = [
@ -54,30 +44,11 @@ STRATEGIES = [


@cache
-def get_github() -> tuple[
-    Optional[tuple[str, str]],
-    Optional[github.Github],
-    Optional[github.InputGitAuthor],
-]:
+def get_github() -> tuple[Optional[tuple[str, str]], Optional[github.Github], Optional[github.InputGitAuthor]]:
    try:
-        github_login = (
-            (REPO_APPS_ROOT / ".github_login")
-            .open("r", encoding="utf-8")
-            .read()
-            .strip()
-        )
-        github_token = (
-            (REPO_APPS_ROOT / ".github_token")
-            .open("r", encoding="utf-8")
-            .read()
-            .strip()
-        )
-        github_email = (
-            (REPO_APPS_ROOT / ".github_email")
-            .open("r", encoding="utf-8")
-            .read()
-            .strip()
-        )
+        github_login = (REPO_APPS_ROOT / ".github_login").open("r", encoding="utf-8").read().strip()
+        github_token = (REPO_APPS_ROOT / ".github_token").open("r", encoding="utf-8").read().strip()
+        github_email = (REPO_APPS_ROOT / ".github_email").open("r", encoding="utf-8").read().strip()

        auth = (github_login, github_token)
        github_api = github.Github(github_token)
@ -125,9 +96,7 @@ class LocalOrRemoteRepo:
            if not self.manifest_path.exists():
                raise RuntimeError(f"{app.name}: manifest.toml doesnt exists?")
            # app is in fact a path
-            self.manifest_raw = (
-                (app / "manifest.toml").open("r", encoding="utf-8").read()
-            )
+            self.manifest_raw = (app / "manifest.toml").open("r", encoding="utf-8").read()

        elif isinstance(app, str):
            # It's remote
@ -218,9 +187,7 @@ class AppAutoUpdater:

        self.main_upstream = self.manifest.get("upstream", {}).get("code")

-    def run(
-        self, edit: bool = False, commit: bool = False, pr: bool = False
-    ) -> tuple[State, str, str, str]:
+    def run(self, edit: bool = False, commit: bool = False, pr: bool = False) -> tuple[State, str, str, str]:
        state = State.up_to_date
        main_version = ""
        pr_url = ""
@ -245,11 +212,7 @@ class AppAutoUpdater:
                    commit_msg += f"\n{msg}"

            self.repo.manifest_raw = self.replace_version_and_asset_in_manifest(
-                self.repo.manifest_raw,
-                version,
-                assets,
-                infos,
-                is_main=source == "main",
+                self.repo.manifest_raw, version, assets, infos, is_main=source == "main",
            )

        if state == State.up_to_date:
@ -283,9 +246,7 @@ class AppAutoUpdater:
        return (state, self.current_version, main_version, pr_url)

    @staticmethod
-    def relevant_versions(
-        tags: list[str], app_id: str, version_regex: Optional[str]
-    ) -> tuple[str, str]:
+    def relevant_versions(tags: list[str], app_id: str, version_regex: Optional[str]) -> tuple[str, str]:

        def apply_version_regex(tag: str) -> Optional[str]:
            # First preprocessing according to the manifest version_regex…
@ -294,9 +255,7 @@ class AppAutoUpdater:
                if match is None:
                    return None
                # Basically: either groupdict if named capture gorups, sorted by names, or groups()
-                tag = ".".join(
-                    dict(sorted(match.groupdict().items())).values() or match.groups()
-                )
+                tag = ".".join(dict(sorted(match.groupdict().items())).values() or match.groups())

            # Then remove leading v
            tag = tag.lstrip("v")
@ -305,9 +264,7 @@ class AppAutoUpdater:
        def version_numbers(tag: str) -> Optional[tuple[int, ...]]:
            filter_keywords = ["start", "rc", "beta", "alpha"]
            if any(keyword in tag for keyword in filter_keywords):
-                logging.debug(
-                    f"Tag {tag} contains filtered keyword from {filter_keywords}."
-                )
+                logging.debug(f"Tag {tag} contains filtered keyword from {filter_keywords}.")
                return None

            t_to_check = tag
@ -345,9 +302,7 @@ class AppAutoUpdater:
    def tag_to_int_tuple(tag: str) -> tuple[int, ...]:
        tag = tag.lstrip("v").replace("-", ".").rstrip(".")
        int_tuple = tag.split(".")
-        assert all(
-            i.isdigit() for i in int_tuple
-        ), f"Cant convert {tag} to int tuple :/"
+        assert all(i.isdigit() for i in int_tuple), f"Cant convert {tag} to int tuple :/"
        return tuple(int(i) for i in int_tuple)

    @staticmethod
@ -362,9 +317,8 @@ class AppAutoUpdater:
        except Exception as e:
            raise RuntimeError(f"Failed to compute sha256 for {url} : {e}") from e

-    def get_source_update(
-        self, name: str, infos: dict[str, Any]
-    ) -> Optional[tuple[str, Union[str, dict[str, str]], str]]:
+    def get_source_update(self, name: str, infos: dict[str, Any]
+                          ) -> Optional[tuple[str, Union[str, dict[str, str]], str]]:
        autoupdate = infos.get("autoupdate")
        if autoupdate is None:
            return None
@ -373,9 +327,7 @@ class AppAutoUpdater:
        asset = autoupdate.get("asset", "tarball")
        strategy = autoupdate.get("strategy")
        if strategy not in STRATEGIES:
-            raise ValueError(
-                f"Unknown update strategy '{strategy}' for '{name}', expected one of {STRATEGIES}"
-            )
+            raise ValueError(f"Unknown update strategy '{strategy}' for '{name}', expected one of {STRATEGIES}")

        result = self.get_latest_version_and_asset(strategy, asset, autoupdate)
        if result is None:
@ -395,22 +347,14 @@ class AppAutoUpdater:
                print("Up to date")
                return None
            try:
-                if self.tag_to_int_tuple(self.current_version) > self.tag_to_int_tuple(
-                    new_version
-                ):
-                    print(
-                        "Up to date (current version appears more recent than newest version found)"
-                    )
+                if self.tag_to_int_tuple(self.current_version) > self.tag_to_int_tuple(new_version):
+                    print("Up to date (current version appears more recent than newest version found)")
                    return None
            except (AssertionError, ValueError):
                pass

-        if (
-            isinstance(assets, dict)
-            and isinstance(infos.get("url"), str)
-            or isinstance(assets, str)
-            and not isinstance(infos.get("url"), str)
-        ):
+        if isinstance(assets, dict) and isinstance(infos.get("url"), str) or \
+           isinstance(assets, str) and not isinstance(infos.get("url"), str):
            raise RuntimeError(
                "It looks like there's an inconsistency between the old asset list and the new ones... "
                "One is arch-specific, the other is not... Did you forget to define arch-specific regexes? "
@ -420,9 +364,7 @@ class AppAutoUpdater:
        if isinstance(assets, str) and infos["url"] == assets:
            print(f"URL for asset {name} is up to date")
            return None
-        if isinstance(assets, dict) and assets == {
-            k: infos[k]["url"] for k in assets.keys()
-        }:
+        if isinstance(assets, dict) and assets == {k: infos[k]["url"] for k in assets.keys()}:
            print(f"URLs for asset {name} are up to date")
            return None
        print(f"Update needed for {name}")
@ -434,26 +376,21 @@ class AppAutoUpdater:
            name: url for name, url in assets.items() if re.match(regex, name)
        }
        if not matching_assets:
-            raise RuntimeError(
-                f"No assets matching regex '{regex}' in {list(assets.keys())}"
-            )
+            raise RuntimeError(f"No assets matching regex '{regex}' in {list(assets.keys())}")
        if len(matching_assets) > 1:
-            raise RuntimeError(
-                f"Too many assets matching regex '{regex}': {matching_assets}"
-            )
+            raise RuntimeError(f"Too many assets matching regex '{regex}': {matching_assets}")
        return next(iter(matching_assets.items()))

-    def get_latest_version_and_asset(
-        self, strategy: str, asset: Union[str, dict], autoupdate
-    ) -> Optional[tuple[str, Union[str, dict[str, str]], str]]:
+    def get_latest_version_and_asset(self, strategy: str, asset: Union[str, dict], autoupdate
+                                     ) -> Optional[tuple[str, Union[str, dict[str, str]], str]]:
        upstream = autoupdate.get("upstream", self.main_upstream).strip("/")
        version_re = autoupdate.get("version_regex", None)
        _, remote_type, revision_type = strategy.split("_")

        api: Union[GithubAPI, GitlabAPI, GiteaForgejoAPI]
        if remote_type == "github":
-            assert upstream and upstream.startswith(
-                "https://github.com/"
+            assert (
+                upstream and upstream.startswith("https://github.com/")
            ), f"When using strategy {strategy}, having a defined upstream code repo on github.com is required"
            api = GithubAPI(upstream, auth=get_github()[0])
        if remote_type == "gitlab":
@ -467,9 +404,7 @@ class AppAutoUpdater:
                for release in api.releases()
                if not release["draft"] and not release["prerelease"]
            }
-            latest_version_orig, latest_version = self.relevant_versions(
-                list(releases.keys()), self.app_id, version_re
-            )
+            latest_version_orig, latest_version = self.relevant_versions(list(releases.keys()), self.app_id, version_re)
            latest_release = releases[latest_version_orig]
            latest_assets = {
                a["name"]: a["browser_download_url"]
@ -490,9 +425,7 @@ class AppAutoUpdater:
                    _, url = self.find_matching_asset(latest_assets, asset)
                    return latest_version, url, latest_release_html_url
                except RuntimeError as e:
-                    raise RuntimeError(
-                        f"{e}.\nFull release details on {latest_release_html_url}."
-                    ) from e
+                    raise RuntimeError(f"{e}.\nFull release details on {latest_release_html_url}.") from e

            if isinstance(asset, dict):
                new_assets = {}
@ -501,50 +434,34 @@ class AppAutoUpdater:
                        _, url = self.find_matching_asset(latest_assets, asset_regex)
                        new_assets[asset_name] = url
                    except RuntimeError as e:
-                        raise RuntimeError(
-                            f"{e}.\nFull release details on {latest_release_html_url}."
-                        ) from e
+                        raise RuntimeError(f"{e}.\nFull release details on {latest_release_html_url}.") from e
                return latest_version, new_assets, latest_release_html_url

            return None

        if revision_type == "tag":
            if asset != "tarball":
-                raise ValueError(
-                    "For the latest tag strategies, only asset = 'tarball' is supported"
-                )
+                raise ValueError("For the latest tag strategies, only asset = 'tarball' is supported")
            tags = [t["name"] for t in api.tags()]
-            latest_version_orig, latest_version = self.relevant_versions(
-                tags, self.app_id, version_re
-            )
+            latest_version_orig, latest_version = self.relevant_versions(tags, self.app_id, version_re)
            latest_tarball = api.url_for_ref(latest_version_orig, RefType.tags)
            return latest_version, latest_tarball, ""

        if revision_type == "commit":
            if asset != "tarball":
-                raise ValueError(
-                    "For the latest commit strategies, only asset = 'tarball' is supported"
-                )
+                raise ValueError("For the latest commit strategies, only asset = 'tarball' is supported")
            commits = api.commits()
            latest_commit = commits[0]
            latest_tarball = api.url_for_ref(latest_commit["sha"], RefType.commits)
            # Let's have the version as something like "2023.01.23"
-            latest_commit_date = datetime.strptime(
-                latest_commit["commit"]["author"]["date"][:10], "%Y-%m-%d"
-            )
+            latest_commit_date = datetime.strptime(latest_commit["commit"]["author"]["date"][:10], "%Y-%m-%d")
            version_format = autoupdate.get("force_version", "%Y.%m.%d")
            latest_version = latest_commit_date.strftime(version_format)
            return latest_version, latest_tarball, ""
        return None

-    def replace_version_and_asset_in_manifest(
-        self,
-        content: str,
-        new_version: str,
-        new_assets_urls: Union[str, dict],
-        current_assets: dict,
-        is_main: bool,
-    ):
+    def replace_version_and_asset_in_manifest(self, content: str, new_version: str, new_assets_urls: Union[str, dict],
+                                              current_assets: dict, is_main: bool):
        replacements = []
        if isinstance(new_assets_urls, str):
            replacements = [
@ -554,21 +471,16 @@ class AppAutoUpdater:
        if isinstance(new_assets_urls, dict):
            replacements = [
                repl
-                for key, url in new_assets_urls.items()
-                for repl in (
+                for key, url in new_assets_urls.items() for repl in (
                    (current_assets[key]["url"], url),
-                    (current_assets[key]["sha256"], self.sha256_of_remote_file(url)),
+                    (current_assets[key]["sha256"], self.sha256_of_remote_file(url))
                )
            ]

        if is_main:
-
            def repl(m: re.Match) -> str:
                return m.group(1) + new_version + '~ynh1"'
-
-            content = re.sub(
-                r"(\s*version\s*=\s*[\"\'])([^~\"\']+)(\~ynh\d+[\"\'])", repl, content
-            )
+            content = re.sub(r"(\s*version\s*=\s*[\"\'])([^~\"\']+)(\~ynh\d+[\"\'])", repl, content)

        for old, new in replacements:
            content = content.replace(old, new)
@ -626,41 +538,22 @@ def run_autoupdate_for_multiprocessing(data) -> tuple[str, tuple[State, str, str
    except Exception:
        log_str = stdoutswitch.reset()
        import traceback
-
        t = traceback.format_exc()
        return (app, (State.failure, log_str, str(t), ""))


 def main() -> None:
    parser = argparse.ArgumentParser()
-    parser.add_argument(
-        "apps",
-        nargs="*",
-        type=Path,
-        help="If not passed, the script will run on the catalog. Github keys required.",
-    )
-    parser.add_argument(
-        "--edit",
-        action=argparse.BooleanOptionalAction,
-        default=True,
-        help="Edit the local files",
-    )
-    parser.add_argument(
-        "--commit",
-        action=argparse.BooleanOptionalAction,
-        default=False,
-        help="Create a commit with the changes",
-    )
-    parser.add_argument(
-        "--pr",
-        action=argparse.BooleanOptionalAction,
-        default=False,
-        help="Create a pull request with the changes",
-    )
+    parser.add_argument("apps", nargs="*", type=Path,
+                        help="If not passed, the script will run on the catalog. Github keys required.")
+    parser.add_argument("--edit", action=argparse.BooleanOptionalAction, default=True,
+                        help="Edit the local files")
+    parser.add_argument("--commit", action=argparse.BooleanOptionalAction, default=False,
+                        help="Create a commit with the changes")
+    parser.add_argument("--pr", action=argparse.BooleanOptionalAction, default=False,
+                        help="Create a pull request with the changes")
    parser.add_argument("--paste", action="store_true")
-    parser.add_argument(
-        "-j", "--processes", type=int, default=multiprocessing.cpu_count()
-    )
+    parser.add_argument("-j", "--processes", type=int, default=multiprocessing.cpu_count())
    args = parser.parse_args()

    appslib.logging_sender.enable()
@ -679,10 +572,8 @@ def main() -> None:
    apps_failed = {}

    with multiprocessing.Pool(processes=args.processes) as pool:
-        tasks = pool.imap(
-            run_autoupdate_for_multiprocessing,
-            ((app, args.edit, args.commit, args.pr) for app in apps),
-        )
+        tasks = pool.imap(run_autoupdate_for_multiprocessing,
+                          ((app, args.edit, args.commit, args.pr) for app in apps))
        for app, result in tqdm.tqdm(tasks, total=len(apps), ascii=" ·#"):
            state, current_version, main_version, pr_url = result
            if state == State.up_to_date:
@ -701,9 +592,7 @@ def main() -> None:
        matrix_message += f"\n- {len(apps_already)} pending update PRs"
    for app, info in apps_already.items():
        paste_message += f"\n- {app}"
-        paste_message += (
-            f" ({info[0]} -> {info[1]})" if info[1] else " (app version did not change)"
-        )
+        paste_message += f" ({info[0]} -> {info[1]})" if info[1] else " (app version did not change)"
        if info[2]:
            paste_message += f" see {info[2]}"

@ -712,9 +601,7 @@ def main() -> None:
        matrix_message += f"\n- {len(apps_updated)} new apps PRs"
    for app, info in apps_updated.items():
        paste_message += f"\n- {app}"
-        paste_message += (
-            f" ({info[0]} -> {info[1]})" if info[1] else " (app version did not change)"
-        )
+        paste_message += f" ({info[0]} -> {info[1]})" if info[1] else " (app version did not change)"
        if info[2]:
            paste_message += f" see {info[2]}"

--- a/tools/autoupdate_app_sources/rest_api.py
+++ b/tools/autoupdate_app_sources/rest_api.py
@ -15,10 +15,11 @@ class RefType(Enum):
 class GithubAPI:
    def __init__(self, upstream: str, auth: Optional[tuple[str, str]] = None):
        self.upstream = upstream
-        self.upstream_repo = upstream.replace("https://github.com/", "").strip("/")
+        self.upstream_repo = upstream.replace("https://github.com/", "")\
+            .strip("/")
        assert (
-            len(self.upstream_repo.split("/")) == 2
-        ), f"'{upstream}' doesn't seem to be a github repository ?"
+                len(self.upstream_repo.split("/")) == 2
+            ), f"'{upstream}' doesn't seem to be a github repository ?"
        self.auth = auth

    def internal_api(self, uri: str) -> Any:
@ -73,12 +74,7 @@ class GitlabAPI:
            # Second chance for some buggy gitlab instances...
            name = self.project_path.split("/")[-1]
            projects = self.internal_api(f"projects?search={name}")
-            project = next(
-                filter(
-                    lambda x: x.get("path_with_namespace") == self.project_path,
-                    projects,
-                )
-            )
+            project = next(filter(lambda x: x.get("path_with_namespace") == self.project_path, projects))

        assert isinstance(project, dict)
        project_id = project.get("id", None)
@ -99,11 +95,13 @@ class GitlabAPI:
        return [
            {
                "sha": commit["id"],
-                "commit": {"author": {"date": commit["committed_date"]}},
+                "commit": {
+                    "author": {
+                        "date": commit["committed_date"]
+                    }
+                }
            }
-            for commit in self.internal_api(
-                f"projects/{self.project_id}/repository/commits"
-            )
+            for commit in self.internal_api(f"projects/{self.project_id}/repository/commits")
        ]

    def releases(self) -> list[dict[str, Any]]:
@ -116,21 +114,16 @@ class GitlabAPI:
                "prerelease": False,
                "draft": False,
                "html_url": release["_links"]["self"],
-                "assets": [
-                    {
-                        "name": asset["name"],
-                        "browser_download_url": asset["direct_asset_url"],
-                    }
-                    for asset in release["assets"]["links"]
-                ],
-            }
+                "assets": [{
+                    "name": asset["name"],
+                    "browser_download_url": asset["direct_asset_url"]
+                    } for asset in release["assets"]["links"]],
+                }
            for source in release["assets"]["sources"]:
-                r["assets"].append(
-                    {
-                        "name": f"source.{source['format']}",
-                        "browser_download_url": source["url"],
-                    }
-                )
+                r["assets"].append({
+                    "name": f"source.{source['format']}",
+                    "browser_download_url": source['url']
+                })
            retval.append(r)

        return retval
--- a/tools/autoupdater-upgrader/autoupdater-upgrader.py
+++ b/tools/autoupdater-upgrader/autoupdater-upgrader.py
@ -9,7 +9,6 @@ import urllib.request

 import github
 from github import Github
-
 # Debug
 from rich.traceback import install

@ -25,25 +24,23 @@ install(width=150, show_locals=True, locals_max_length=None, locals_max_string=N
 g = Github(open(".github_token").read().strip())

 # Path to the file to be updated
-path = ".github/workflows/updater.yml"
+path=".github/workflows/updater.yml"

 # Title of the PR
-title = "[autopatch] Upgrade auto-updater"
+title="[autopatch] Upgrade auto-updater"

 # Body of the PR message
-body = """
+body="""
 Auto-updater actions need upgrading to continue working:
 - actions/checkout@v3
 - peter-evans/create-pull-request@v4
 """

 # Author of the commit
-author = github.InputGitAuthor(
-    open(".github_login").read().strip(), open(".github_email").read().strip()
-)
+author=github.InputGitAuthor(open(".github_login").read().strip(), open(".github_email").read().strip())

 # Name of the branch created for the PR
-new_branch = "upgrade-auto-updater"
+new_branch="upgrade-auto-updater"

 #####
 #
@ -51,7 +48,7 @@ new_branch = "upgrade-auto-updater"
 #
 #####

-with open("processed.txt") as f:
+with open('processed.txt') as f:
    processed = f.read().splitlines()

 #####
@ -64,7 +61,7 @@ u = g.get_user("yunohost-bot")
 org = g.get_organization("yunohost-apps")

 # For each repositories belonging to the bot (user `u`)
-i = 0
+i=0
 for repo in org.get_repos():
    if repo.full_name not in processed:

@ -76,64 +73,50 @@ for repo in org.get_repos():

        # Make sure the repository has an auto-updater
        try:
-            repo.get_contents(path, ref="refs/heads/" + base_branch)
+            repo.get_contents(path, ref="refs/heads/"+base_branch)
        except:
-            with open("processed.txt", "a") as pfile:
-                pfile.write(repo.full_name + "\n")
+            with open('processed.txt', 'a') as pfile:
+                pfile.write(repo.full_name+'\n')
            time.sleep(1.5)
            continue

        # Process the repo
-        print("Processing " + repo.full_name)
+        print("Processing "+repo.full_name)

        try:
            # Get the commit base for the new branch, and create it
            commit_sha = repo.get_branch(base_branch).commit.sha
-            new_branch_ref = repo.create_git_ref(
-                ref="refs/heads/" + new_branch, sha=commit_sha
-            )
+            new_branch_ref = repo.create_git_ref(ref="refs/heads/"+new_branch, sha=commit_sha)
        except:
-            new_branch_ref = repo.get_git_ref(ref="heads/" + new_branch)
+            new_branch_ref = repo.get_git_ref(ref="heads/"+new_branch)

        # Get current file contents
        contents = repo.get_contents(path, ref=new_branch_ref.ref)

        # Update the file
        updater_yml = contents.decoded_content.decode("unicode_escape")
-        updater_yml = re.sub(
-            r"(?m)uses: actions/checkout@v[\d]+",
-            "uses: actions/checkout@v3",
-            updater_yml,
-        )
-        updater_yml = re.sub(
-            r"(?m)uses: peter-evans/create-pull-request@v[\d]+",
-            "uses: peter-evans/create-pull-request@v4",
-            updater_yml,
-        )
-        updated = repo.update_file(
-            contents.path,
-            message=title,
-            content=updater_yml,
-            sha=contents.sha,
-            branch=new_branch,
-            author=author,
-        )
+        updater_yml = re.sub(r'(?m)uses: actions/checkout@v[\d]+', "uses: actions/checkout@v3", updater_yml)
+        updater_yml = re.sub(r'(?m)uses: peter-evans/create-pull-request@v[\d]+', "uses: peter-evans/create-pull-request@v4", updater_yml)
+        updated = repo.update_file(contents.path,
+                                    message=title,
+                                    content=updater_yml,
+                                    sha=contents.sha,
+                                    branch=new_branch,
+                                    author=author)

        # Wait a bit to preserve the API rate limit
        time.sleep(1.5)

        # Open the PR
-        pr = repo.create_pull(
-            title="Upgrade auto-updater", body=body, head=new_branch, base=base_branch
-        )
+        pr = repo.create_pull(title="Upgrade auto-updater", body=body, head=new_branch, base=base_branch)

-        print(repo.full_name + " updated with PR #" + str(pr.id))
-        i = i + 1
+        print(repo.full_name+" updated with PR #"+ str(pr.id))
+        i=i+1

        # Wait a bit to preserve the API rate limit
        time.sleep(1.5)

-        with open("processed.txt", "a") as pfile:
-            pfile.write(repo.full_name + "\n")
+        with open('processed.txt', 'a') as pfile:
+            pfile.write(repo.full_name+'\n')

-print("Done. " + str(i) + " repos processed")
+print("Done. "+str(i)+" repos processed")
--- a/tools/bot-repo-cleanup/cleanup.py
+++ b/tools/bot-repo-cleanup/cleanup.py
@ -10,22 +10,17 @@ u = g.get_user("yunohost-bot")

 # Let's build a minimalistic summary table
 print("| Repository ".ljust(22) + " | Decision |")
-print("| ".ljust(22, "-") + " | -------- |")
+print("| ".ljust(22, '-')       + " | -------- |")

 # For each repositories belonging to the bot (user `u`)
 for repo in u.get_repos():
    # Proceed iff the repository is a fork (`parent` key is set) of a repository in our apps organization
-    if repo.parent.full_name.split("/")[0] != "YunoHost-Apps":
-        print("| " + repo.name.ljust(20) + " | Skipping |")
+    if repo.parent.full_name.split('/')[0] != "YunoHost-Apps":
+        print("| "+repo.name.ljust(20) + " | Skipping |")
    else:
        # If none of the PRs are opened by the bot, delete the repository
-        if not any(
-            [
-                (pr.user == u)
-                for pr in list(repo.parent.get_pulls(state="open", sort="created"))
-            ]
-        ):
-            print("| " + repo.name.ljust(20) + " | Deleting |")
+        if not any([ (pr.user == u) for pr in list(repo.parent.get_pulls(state='open', sort='created')) ]):
+            print("| "+repo.name.ljust(20) + " | Deleting |")
            repo.delete()
        else:
-            print("| " + repo.name.ljust(20) + " | Keeping  |")
+            print("| "+repo.name.ljust(20) + " | Keeping  |")
--- a/tools/catalog_linter.py
+++ b/tools/catalog_linter.py
@ -6,29 +6,20 @@ from difflib import SequenceMatcher
 from typing import Any, Dict, Generator, List, Tuple

 import jsonschema
-from appslib.utils import (
-    REPO_APPS_ROOT,  # pylint: disable=import-error
-    get_antifeatures,
-    get_catalog,
-    get_categories,
-    get_graveyard,
-    get_wishlist,
-)
+from appslib.utils import (REPO_APPS_ROOT,  # pylint: disable=import-error
+                           get_antifeatures, get_catalog, get_categories,
+                           get_graveyard, get_wishlist)


 def validate_schema() -> Generator[str, None, None]:
-    with open(
-        REPO_APPS_ROOT / "schemas" / "apps.toml.schema.json", encoding="utf-8"
-    ) as file:
+    with open(REPO_APPS_ROOT / "schemas" / "apps.toml.schema.json", encoding="utf-8") as file:
        apps_catalog_schema = json.load(file)
    validator = jsonschema.Draft202012Validator(apps_catalog_schema)
    for error in validator.iter_errors(get_catalog()):
        yield f"at .{'.'.join(error.path)}: {error.message}"


-def check_app(
-    app: str, infos: Dict[str, Any]
-) -> Generator[Tuple[str, bool], None, None]:
+def check_app(app: str, infos: Dict[str, Any]) -> Generator[Tuple[str, bool], None, None]:
    if "state" not in infos:
        yield "state is missing", True
        return
--- a/tools/list_builder.py
+++ b/tools/list_builder.py
@ -21,15 +21,10 @@ from git import Repo
 import appslib.logging_sender  # pylint: disable=import-error
 from app_caches import app_cache_folder  # pylint: disable=import-error
 from app_caches import apps_cache_update_all  # pylint: disable=import-error
-from appslib.utils import (
-    REPO_APPS_ROOT,  # pylint: disable=import-error
-    get_antifeatures,
-    get_catalog,
-    get_categories,
-)
-from packaging_v2.convert_v1_manifest_to_v2_for_catalog import (
-    convert_v1_manifest_to_v2_for_catalog,
-)  # pylint: disable=import-error
+from appslib.utils import (REPO_APPS_ROOT,  # pylint: disable=import-error
+                           get_antifeatures, get_catalog, get_categories)
+from packaging_v2.convert_v1_manifest_to_v2_for_catalog import \
+    convert_v1_manifest_to_v2_for_catalog  # pylint: disable=import-error

 now = time.time()

@ -42,7 +37,7 @@ def categories_list():
        infos["id"] = category_id
        for subtag_id, subtag_infos in infos.get("subtags", {}).items():
            subtag_infos["id"] = subtag_id
-        infos["subtags"] = list(infos.get("subtags", {}).values())
+        infos["subtags"] = list(infos.get('subtags', {}).values())
    return list(new_categories.values())


@ -59,7 +54,6 @@ def antifeatures_list():
 # Actual list build management #
 ################################

-
 def __build_app_dict(data) -> Optional[tuple[str, dict[str, Any]]]:
    name, info = data
    try:
@ -99,17 +93,13 @@ def write_catalog_v2(base_catalog, target_dir: Path) -> None:

    target_file = target_dir / "apps.json"
    target_file.parent.mkdir(parents=True, exist_ok=True)
-    target_file.open("w", encoding="utf-8").write(
-        json.dumps(full_catalog, sort_keys=True)
-    )
+    target_file.open("w", encoding="utf-8").write(json.dumps(full_catalog, sort_keys=True))


 def write_catalog_v3(base_catalog, target_dir: Path) -> None:
    result_dict_with_manifest_v2 = copy.deepcopy(base_catalog)
    for app in result_dict_with_manifest_v2.values():
-        packaging_format = float(
-            str(app["manifest"].get("packaging_format", "")).strip() or "0"
-        )
+        packaging_format = float(str(app["manifest"].get("packaging_format", "")).strip() or "0")
        if packaging_format < 2:
            app["manifest"] = convert_v1_manifest_to_v2_for_catalog(app["manifest"])

@ -127,12 +117,7 @@ def write_catalog_v3(base_catalog, target_dir: Path) -> None:
        appid = appid.lower()
        logo_source = REPO_APPS_ROOT / "logos" / f"{appid}.png"
        if logo_source.exists():
-            logo_hash = (
-                subprocess.check_output(["sha256sum", logo_source])
-                .strip()
-                .decode("utf-8")
-                .split()[0]
-            )
+            logo_hash = subprocess.check_output(["sha256sum", logo_source]).strip().decode("utf-8").split()[0]
            shutil.copyfile(logo_source, logos_dir / f"{logo_hash}.png")
            # FIXME: implement something to cleanup old logo stuf in the builds/.../logos/ folder somehow
        else:
@ -147,9 +132,7 @@ def write_catalog_v3(base_catalog, target_dir: Path) -> None:

    target_file = target_dir / "apps.json"
    target_file.parent.mkdir(parents=True, exist_ok=True)
-    target_file.open("w", encoding="utf-8").write(
-        json.dumps(full_catalog, sort_keys=True)
-    )
+    target_file.open("w", encoding="utf-8").write(json.dumps(full_catalog, sort_keys=True))


 def write_catalog_doc(base_catalog, target_dir: Path) -> None:
@ -177,13 +160,14 @@ def write_catalog_doc(base_catalog, target_dir: Path) -> None:
        for k, v in base_catalog.items()
        if v["state"] == "working"
    }
-    full_catalog = {"apps": result_dict_doc, "categories": categories_list()}
+    full_catalog = {
+        "apps": result_dict_doc,
+        "categories": categories_list()
+    }

    target_file = target_dir / "apps.json"
    target_file.parent.mkdir(parents=True, exist_ok=True)
-    target_file.open("w", encoding="utf-8").write(
-        json.dumps(full_catalog, sort_keys=True)
-    )
+    target_file.open("w", encoding="utf-8").write(json.dumps(full_catalog, sort_keys=True))


 def build_app_dict(app, infos):
@ -193,38 +177,15 @@ def build_app_dict(app, infos):

    repo = Repo(this_app_cache)

-    commits_in_apps_json = (
-        Repo(REPO_APPS_ROOT)
-        .git.log(
-            "-S",
-            f'"{app}"',
-            "--first-parent",
-            "--reverse",
-            "--date=unix",
-            "--format=%cd",
-            "--",
-            "apps.json",
-        )
-        .split("\n")
-    )
+    commits_in_apps_json = Repo(REPO_APPS_ROOT).git.log(
+            "-S", f"\"{app}\"", "--first-parent", "--reverse", "--date=unix",
+            "--format=%cd", "--", "apps.json").split("\n")
    if len(commits_in_apps_json) > 1:
        first_commit = commits_in_apps_json[0]
    else:
-        commits_in_apps_toml = (
-            Repo(REPO_APPS_ROOT)
-            .git.log(
-                "-S",
-                f"[{app}]",
-                "--first-parent",
-                "--reverse",
-                "--date=unix",
-                "--format=%cd",
-                "--",
-                "apps.json",
-                "apps.toml",
-            )
-            .split("\n")
-        )
+        commits_in_apps_toml = Repo(REPO_APPS_ROOT).git.log(
+                "-S", f"[{app}]", "--first-parent", "--reverse", "--date=unix",
+                "--format=%cd", "--", "apps.json", "apps.toml").split("\n")
        first_commit = commits_in_apps_toml[0]

    # Assume the first entry we get (= the oldest) is the time the app was added
@ -243,18 +204,14 @@ def build_app_dict(app, infos):
        try:
            _ = repo.commit(infos["revision"])
        except ValueError as err:
-            raise RuntimeError(
-                f"Revision ain't in history ? {infos['revision']}"
-            ) from err
+            raise RuntimeError(f"Revision ain't in history ? {infos['revision']}") from err

    # Find timestamp corresponding to that commit
    timestamp = repo.commit(infos["revision"]).committed_date

    # Build the dict with all the infos
    if (this_app_cache / "manifest.toml").exists():
-        manifest = toml.load(
-            (this_app_cache / "manifest.toml").open("r"), _dict=OrderedDict
-        )
+        manifest = toml.load((this_app_cache / "manifest.toml").open("r"), _dict=OrderedDict)
    else:
        manifest = json.load((this_app_cache / "manifest.json").open("r"))

@ -270,45 +227,27 @@ def build_app_dict(app, infos):
        "manifest": manifest,
        "state": infos["state"],
        "level": infos.get("level", "?"),
-        "maintained": "package-not-maintained" not in infos.get("antifeatures", []),
+        "maintained": 'package-not-maintained' not in infos.get('antifeatures', []),
        "high_quality": infos.get("high_quality", False),
        "featured": infos.get("featured", False),
        "category": infos.get("category", None),
        "subtags": infos.get("subtags", []),
        "potential_alternative_to": infos.get("potential_alternative_to", []),
        "antifeatures": list(
-            set(
-                list(manifest.get("antifeatures", {}).keys())
-                + infos.get("antifeatures", [])
-            )
+            set(list(manifest.get("antifeatures", {}).keys()) + infos.get("antifeatures", []))
        ),
    }


 def main() -> None:
    parser = argparse.ArgumentParser()
-    parser.add_argument(
-        "target_dir",
-        type=Path,
-        nargs="?",
-        default=REPO_APPS_ROOT / "builds" / "default",
-        help="The directory to write the catalogs to",
-    )
-    parser.add_argument(
-        "-j",
-        "--jobs",
-        type=int,
-        default=multiprocessing.cpu_count(),
-        metavar="N",
-        help="Allow N threads to run in parallel",
-    )
-    parser.add_argument(
-        "-c",
-        "--update-cache",
-        action=argparse.BooleanOptionalAction,
-        default=True,
-        help="Update the apps cache",
-    )
+    parser.add_argument("target_dir", type=Path, nargs="?",
+                        default=REPO_APPS_ROOT / "builds" / "default",
+                        help="The directory to write the catalogs to")
+    parser.add_argument("-j", "--jobs", type=int, default=multiprocessing.cpu_count(), metavar="N",
+                        help="Allow N threads to run in parallel")
+    parser.add_argument("-c", "--update-cache", action=argparse.BooleanOptionalAction, default=True,
+                        help="Update the apps cache")
    args = parser.parse_args()

    appslib.logging_sender.enable()
--- a/tools/packaging_v2/convert_app_to_packaging_v2.py
+++ b/tools/packaging_v2/convert_app_to_packaging_v2.py
@ -9,7 +9,11 @@ from glob import glob


 def check_output(cmd):
-    return subprocess.check_output(cmd, shell=True).decode("utf-8").strip()
+    return (
+        subprocess.check_output(cmd, shell=True)
+        .decode("utf-8")
+        .strip()
+    )


 def convert_app_sources(folder):
@ -31,13 +35,7 @@ def convert_app_sources(folder):
            "sha256": D["sum"],
        }

-        if D.get("format", "tar.gz") not in [
-            "zip",
-            "tar.gz",
-            "tar.xz",
-            "tgz",
-            "tar.bz2",
-        ]:
+        if D.get("format", "tar.gz") not in ["zip", "tar.gz", "tar.xz", "tgz", "tar.bz2"]:
            new_D["format"] = D["format"]
            if "filename" in D:
                new_D["rename"] = D["filename"]
@ -117,12 +115,12 @@ def _convert_v1_manifest_to_v2(app_path):
        "sso": "?",
        "disk": "50M",
        "ram.build": "50M",
-        "ram.runtime": "50M",
+        "ram.runtime": "50M"
    }

    maintainers = manifest.get("maintainer", {})
    if isinstance(maintainers, list):
-        maintainers = [m["name"] for m in maintainers]
+        maintainers = [m['name'] for m in maintainers]
    else:
        maintainers = [maintainers["name"]] if maintainers.get("name") else []

@ -132,30 +130,15 @@ def _convert_v1_manifest_to_v2(app_path):
    manifest["install"] = {}
    for question in install_questions:
        name = question.pop("name")
-        if "ask" in question and name in [
-            "domain",
-            "path",
-            "admin",
-            "is_public",
-            "password",
-        ]:
+        if "ask" in question and name in ["domain", "path", "admin", "is_public", "password"]:
            question.pop("ask")
-        if question.get("example") and question.get("type") in [
-            "domain",
-            "path",
-            "user",
-            "boolean",
-            "password",
-        ]:
+        if question.get("example") and question.get("type") in ["domain", "path", "user", "boolean", "password"]:
            question.pop("example")

        manifest["install"][name] = question

    # Rename is_public to init_main_permission
-    manifest["install"] = {
-        (k if k != "is_public" else "init_main_permission"): v
-        for k, v in manifest["install"].items()
-    }
+    manifest["install"] = {(k if k != "is_public" else "init_main_permission"): v for k, v in manifest["install"].items()}

    if "init_main_permission" in manifest["install"]:
        manifest["install"]["init_main_permission"]["type"] = "group"
@ -183,16 +166,12 @@ def _convert_v1_manifest_to_v2(app_path):

    # FIXME: Parse ynh_permission_create --permission="admin" --url="/wp-login.php" --additional_urls="/wp-admin.php" --allowed=$admin_wordpress

-    ports = check_output(
-        f"sed -nr 's/(\\w+)=.*ynh_find_port[^0-9]*([0-9]+)\\)/\\1,\\2/p' '{app_path}/scripts/install'"
-    )
+    ports = check_output(f"sed -nr 's/(\\w+)=.*ynh_find_port[^0-9]*([0-9]+)\\)/\\1,\\2/p' '{app_path}/scripts/install'")
    if ports:
        manifest["resources"]["ports"] = {}
        for port in ports.split("\n"):
            name, default = port.split(",")
-            exposed = check_output(
-                f"sed -nr 's/.*yunohost firewall allow .*(TCP|UDP|Both).*${name}/\\1/p' '{app_path}/scripts/install'"
-            )
+            exposed = check_output(f"sed -nr 's/.*yunohost firewall allow .*(TCP|UDP|Both).*${name}/\\1/p' '{app_path}/scripts/install'")
            if exposed == "Both":
                exposed = True

@ -201,9 +180,7 @@ def _convert_v1_manifest_to_v2(app_path):
                name = "main"

            if not default.isdigit():
-                print(
-                    f"Failed to parse '{default}' as a port number ... Will use 12345 instead"
-                )
+                print(f"Failed to parse '{default}' as a port number ... Will use 12345 instead")
                default = 12345

            manifest["resources"]["ports"][f"{name}.default"] = int(default)
@ -211,57 +188,35 @@ def _convert_v1_manifest_to_v2(app_path):
                manifest["resources"]["ports"][f"{name}.exposed"] = exposed

    maybequote = "[\"'\"'\"']?"
-    apt_dependencies = check_output(
-        f"sed -nr 's/.*_dependencies={maybequote}(.*){maybequote}? *$/\\1/p' '{app_path}/scripts/_common.sh' 2>/dev/null | tr -d '\"' | sed 's@ @\\n@g'"
-    )
-    php_version = check_output(
-        f"sed -nr 's/^ *YNH_PHP_VERSION={maybequote}(.*){maybequote}?$/\\1/p' '{app_path}/scripts/_common.sh' 2>/dev/null | tr -d \"\\\"'\""
-    )
+    apt_dependencies = check_output(f"sed -nr 's/.*_dependencies={maybequote}(.*){maybequote}? *$/\\1/p' '{app_path}/scripts/_common.sh' 2>/dev/null | tr -d '\"' | sed 's@ @\\n@g'")
+    php_version = check_output(f"sed -nr 's/^ *YNH_PHP_VERSION={maybequote}(.*){maybequote}?$/\\1/p' '{app_path}/scripts/_common.sh' 2>/dev/null | tr -d \"\\\"'\"")
    if apt_dependencies.strip():
        if php_version:
-            apt_dependencies = apt_dependencies.replace(
-                "${YNH_PHP_VERSION}", php_version
-            )
-        apt_dependencies = ", ".join([d for d in apt_dependencies.split("\n") if d])
+            apt_dependencies = apt_dependencies.replace("${YNH_PHP_VERSION}", php_version)
+        apt_dependencies = ', '.join([d for d in apt_dependencies.split("\n") if d])
        manifest["resources"]["apt"] = {"packages": apt_dependencies}

-    extra_apt_repos = check_output(
-        r"sed -nr 's/.*_extra_app_dependencies.*repo=\"(.*)\".*package=\"(.*)\".*key=\"(.*)\"/\1,\2,\3/p' %s/scripts/install"
-        % app_path
-    )
+    extra_apt_repos = check_output(r"sed -nr 's/.*_extra_app_dependencies.*repo=\"(.*)\".*package=\"(.*)\".*key=\"(.*)\"/\1,\2,\3/p' %s/scripts/install" % app_path)
    if extra_apt_repos:
        for i, extra_apt_repo in enumerate(extra_apt_repos.split("\n")):
            repo, packages, key = extra_apt_repo.split(",")
-            packages = packages.replace("$", "#FIXME#$")
+            packages = packages.replace('$', '#FIXME#$')
            if "apt" not in manifest["resources"]:
                manifest["resources"]["apt"] = {}
            if "extras" not in manifest["resources"]["apt"]:
                manifest["resources"]["apt"]["extras"] = []
-            manifest["resources"]["apt"]["extras"].append(
-                {
-                    "repo": repo,
-                    "key": key,
-                    "packages": packages,
-                }
-            )
+            manifest["resources"]["apt"]["extras"].append({
+                "repo": repo,
+                "key": key,
+                "packages": packages,
+            })

    if os.system(f"grep -q 'ynh_mysql_setup_db' {app_path}/scripts/install") == 0:
        manifest["resources"]["database"] = {"type": "mysql"}
    elif os.system(f"grep -q 'ynh_psql_setup_db' {app_path}/scripts/install") == 0:
        manifest["resources"]["database"] = {"type": "postgresql"}

-    keys_to_keep = [
-        "packaging_format",
-        "id",
-        "name",
-        "description",
-        "version",
-        "maintainers",
-        "upstream",
-        "integration",
-        "install",
-        "resources",
-    ]
+    keys_to_keep = ["packaging_format", "id", "name", "description", "version", "maintainers", "upstream", "integration", "install", "resources"]

    keys_to_del = [key for key in manifest.keys() if key not in keys_to_keep]
    for key in keys_to_del:
@ -291,35 +246,19 @@ def _dump_v2_manifest_as_toml(manifest):
    upstream = table()
    for key, value in manifest["upstream"].items():
        upstream[key] = value
-    upstream["cpe"].comment(
-        "FIXME: optional but recommended if relevant, this is meant to contain the Common Platform Enumeration, which is sort of a standard id for applications defined by the NIST. In particular, Yunohost may use this is in the future to easily track CVE (=security reports) related to apps. The CPE may be obtained by searching here: https://nvd.nist.gov/products/cpe/search. For example, for Nextcloud, the CPE is 'cpe:2.3:a:nextcloud:nextcloud' (no need to include the version number)"
-    )
-    upstream["fund"].comment(
-        "FIXME: optional but recommended (or remove if irrelevant / not applicable). This is meant to be an URL where people can financially support this app, especially when its development is based on volunteers and/or financed by its community. YunoHost may later advertise it in the webadmin."
-    )
+    upstream["cpe"].comment("FIXME: optional but recommended if relevant, this is meant to contain the Common Platform Enumeration, which is sort of a standard id for applications defined by the NIST. In particular, Yunohost may use this is in the future to easily track CVE (=security reports) related to apps. The CPE may be obtained by searching here: https://nvd.nist.gov/products/cpe/search. For example, for Nextcloud, the CPE is 'cpe:2.3:a:nextcloud:nextcloud' (no need to include the version number)")
+    upstream["fund"].comment("FIXME: optional but recommended (or remove if irrelevant / not applicable). This is meant to be an URL where people can financially support this app, especially when its development is based on volunteers and/or financed by its community. YunoHost may later advertise it in the webadmin.")
    toml_manifest["upstream"] = upstream

    integration = table()
    for key, value in manifest["integration"].items():
        integration.add(key, value)
-    integration["architectures"].comment(
-        'FIXME: can be replaced by a list of supported archs using the dpkg --print-architecture nomenclature (amd64/i386/armhf/arm64), for example: ["amd64", "i386"]'
-    )
-    integration["ldap"].comment(
-        'FIXME: replace with true, false, or "not_relevant". Not to confuse with the "sso" key : the "ldap" key corresponds to wether or not a user *can* login on the app using its YunoHost credentials.'
-    )
-    integration["sso"].comment(
-        'FIXME: replace with true, false, or "not_relevant". Not to confuse with the "ldap" key : the "sso" key corresponds to wether or not a user is *automatically logged-in* on the app when logged-in on the YunoHost portal.'
-    )
-    integration["disk"].comment(
-        "FIXME: replace with an **estimate** minimum disk requirement. e.g. 20M, 400M, 1G, ..."
-    )
-    integration["ram.build"].comment(
-        "FIXME: replace with an **estimate** minimum ram requirement. e.g. 50M, 400M, 1G, ..."
-    )
-    integration["ram.runtime"].comment(
-        "FIXME: replace with an **estimate** minimum ram requirement. e.g. 50M, 400M, 1G, ..."
-    )
+    integration["architectures"].comment('FIXME: can be replaced by a list of supported archs using the dpkg --print-architecture nomenclature (amd64/i386/armhf/arm64), for example: ["amd64", "i386"]')
+    integration["ldap"].comment('FIXME: replace with true, false, or "not_relevant". Not to confuse with the "sso" key : the "ldap" key corresponds to wether or not a user *can* login on the app using its YunoHost credentials.')
+    integration["sso"].comment('FIXME: replace with true, false, or "not_relevant". Not to confuse with the "ldap" key : the "sso" key corresponds to wether or not a user is *automatically logged-in* on the app when logged-in on the YunoHost portal.')
+    integration["disk"].comment('FIXME: replace with an **estimate** minimum disk requirement. e.g. 20M, 400M, 1G, ...')
+    integration["ram.build"].comment('FIXME: replace with an **estimate** minimum ram requirement. e.g. 50M, 400M, 1G, ...')
+    integration["ram.runtime"].comment('FIXME: replace with an **estimate** minimum ram requirement. e.g. 50M, 400M, 1G, ...')
    toml_manifest["integration"] = integration

    install = table()
@ -328,11 +267,7 @@ def _dump_v2_manifest_as_toml(manifest):
        install[key].indent(4)

        if key in ["domain", "path", "admin", "is_public", "password"]:
-            install[key].add(
-                comment(
-                    "this is a generic question - ask strings are automatically handled by Yunohost's core"
-                )
-            )
+            install[key].add(comment("this is a generic question - ask strings are automatically handled by Yunohost's core"))

        for lang, value2 in value.get("ask", {}).items():
            install[key].add(f"ask.{lang}", value2)
@ -370,8 +305,8 @@ def _dump_v2_manifest_as_toml(manifest):

    toml_manifest_dump = dumps(toml_manifest)

-    regex = re.compile(r"\"((description|ask|help)\.[a-z]{2})\"")
-    toml_manifest_dump = regex.sub(r"\1", toml_manifest_dump)
+    regex = re.compile(r'\"((description|ask|help)\.[a-z]{2})\"')
+    toml_manifest_dump = regex.sub(r'\1', toml_manifest_dump)
    toml_manifest_dump = toml_manifest_dump.replace('"ram.build"', "ram.build")
    toml_manifest_dump = toml_manifest_dump.replace('"ram.runtime"', "ram.runtime")
    toml_manifest_dump = toml_manifest_dump.replace('"main.url"', "main.url")
@ -389,9 +324,7 @@ def _dump_v2_manifest_as_toml(manifest):

    if "ports" in manifest["resources"]:
        for port_thing in manifest["resources"]["ports"].keys():
-            toml_manifest_dump = toml_manifest_dump.replace(
-                f'"{port_thing}"', f"{port_thing}"
-            )
+            toml_manifest_dump = toml_manifest_dump.replace(f'"{port_thing}"', f"{port_thing}")

    return toml_manifest_dump

@ -462,9 +395,7 @@ def cleanup_scripts_and_conf(folder):
        "^.*ynh_script_progression.*Reloading NGINX web server",
        "^.*ynh_systemd_action --service_name=nginx --action=reload",
    ]
-    patterns_to_remove_in_scripts = [
-        re.compile(f"({p})", re.MULTILINE) for p in patterns_to_remove_in_scripts
-    ]
+    patterns_to_remove_in_scripts = [re.compile(f"({p})", re.MULTILINE) for p in patterns_to_remove_in_scripts]

    replaces = [
        ("path_url", "path"),
@ -473,21 +404,13 @@ def cleanup_scripts_and_conf(folder):
        ("FINALPATH", "INSTALL_DIR"),
        ("datadir", "data_dir"),
        ("DATADIR", "DATA_DIR"),
-        ('--source_id="$architecture"', ""),
-        ('--source_id="$YNH_ARCH"', ""),
-        ("--source_id=app", ""),
-        ('--source_id="app.$architecture"', ""),
+        ('--source_id="$architecture"', ''),
+        ('--source_id="$YNH_ARCH"', ''),
+        ('--source_id=app', ''),
+        ('--source_id="app.$architecture"', ''),
    ]

-    for s in [
-        "_common.sh",
-        "install",
-        "remove",
-        "upgrade",
-        "backup",
-        "restore",
-        "change_url",
-    ]:
+    for s in ["_common.sh", "install", "remove", "upgrade", "backup", "restore", "change_url"]:

        script = f"{folder}/scripts/{s}"

@ -497,18 +420,10 @@ def cleanup_scripts_and_conf(folder):
        content = open(script).read()

        for pattern in patterns_to_remove_in_scripts:
-            if (
-                "^.*ynh_script_progression.*Reloading NGINX web server"
-                in pattern.pattern
-                and s == "restore"
-            ):
+            if "^.*ynh_script_progression.*Reloading NGINX web server" in pattern.pattern and s == "restore":
                # This case is legit
                continue
-            if (
-                "^.*ynh_systemd_action --service_name=nginx --action=reload"
-                in pattern.pattern
-                and s == "restore"
-            ):
+            if "^.*ynh_systemd_action --service_name=nginx --action=reload" in pattern.pattern and s == "restore":
                # This case is legit
                continue
            content = pattern.sub(r"#REMOVEME? \1", content)
@ -521,9 +436,7 @@ def cleanup_scripts_and_conf(folder):
            pattern = re.compile("(^.*nginx.*$)", re.MULTILINE)
            content = pattern.sub(r"#REMOVEME? \1", content)

-            pattern = re.compile(
-                "(^.*ynh_script_progress.*Updat.* NGINX.*conf.*$)", re.MULTILINE
-            )
+            pattern = re.compile("(^.*ynh_script_progress.*Updat.* NGINX.*conf.*$)", re.MULTILINE)
            content = pattern.sub(r"\1\n\nynh_change_url_nginx_config", content)

            pattern = re.compile(r"(ynh_clean_check_starting)", re.MULTILINE)
@ -533,6 +446,7 @@ def cleanup_scripts_and_conf(folder):
            pattern = re.compile(r"(^\s+path=.*$)", re.MULTILINE)
            content = pattern.sub(r"#REMOVEME? \1", content)

+
        open(script, "w").write(content)

    for conf in os.listdir(f"{folder}/conf"):
@ -556,15 +470,15 @@ if __name__ == "__main__":
    parser = argparse.ArgumentParser(
        description="Attempt to automatically convert a v1 YunoHost app to v2 (at least as much as possible) : parse the app scripts to auto-generate the manifest.toml, and remove now-useless lines from the app scripts"
    )
-    parser.add_argument("app_path", help="Path to the app to convert")
+    parser.add_argument(
+        "app_path", help="Path to the app to convert"
+    )

    args = parser.parse_args()

    manifest = _convert_v1_manifest_to_v2(args.app_path)
    with open(args.app_path + "/manifest.toml", "w") as manifest_file:
-        manifest_file.write(
-            "#:schema https://raw.githubusercontent.com/YunoHost/apps/master/schemas/manifest.v2.schema.json\n\n"
-        )
+        manifest_file.write("#:schema https://raw.githubusercontent.com/YunoHost/apps/master/schemas/manifest.v2.schema.json\n\n")
        manifest_file.write(_dump_v2_manifest_as_toml(manifest))

    cleanup_scripts_and_conf(args.app_path)
--- a/tools/packaging_v2/convert_v1_manifest_to_v2_for_catalog.py
+++ b/tools/packaging_v2/convert_v1_manifest_to_v2_for_catalog.py
@ -17,22 +17,18 @@ def convert_v1_manifest_to_v2_for_catalog(manifest):
        manifest["upstream"]["website"] = manifest["url"]

    manifest["integration"] = {
-        "yunohost": manifest.get("requirements", {})
-        .get("yunohost", "")
-        .replace(">", "")
-        .replace("=", "")
-        .replace(" ", ""),
+        "yunohost": manifest.get("requirements", {}).get("yunohost", "").replace(">", "").replace("=", "").replace(" ", ""),
        "architectures": "all",
        "multi_instance": manifest.get("multi_instance", False),
        "ldap": "?",
        "sso": "?",
        "disk": "50M",
-        "ram": {"build": "50M", "runtime": "10M"},
+        "ram": {"build": "50M", "runtime": "10M"}
    }

    maintainers = manifest.get("maintainer", {})
    if isinstance(maintainers, list):
-        maintainers = [m["name"] for m in maintainers]
+        maintainers = [m['name'] for m in maintainers]
    else:
        maintainers = [maintainers["name"]] if maintainers.get("name") else []

@ -43,39 +39,21 @@ def convert_v1_manifest_to_v2_for_catalog(manifest):
    manifest["install"] = {}
    for question in install_questions:
        name = question.pop("name")
-        if "ask" in question and name in [
-            "domain",
-            "path",
-            "admin",
-            "is_public",
-            "password",
-        ]:
+        if "ask" in question and name in ["domain", "path", "admin", "is_public", "password"]:
            question.pop("ask")
-        if question.get("example") and question.get("type") in [
-            "domain",
-            "path",
-            "user",
-            "boolean",
-            "password",
-        ]:
+        if question.get("example") and question.get("type") in ["domain", "path", "user", "boolean", "password"]:
            question.pop("example")

        manifest["install"][name] = question

-    manifest["resources"] = {"system_user": {}, "install_dir": {"alias": "final_path"}}
+    manifest["resources"] = {
+        "system_user": {},
+        "install_dir": {
+            "alias": "final_path"
+        }
+    }

-    keys_to_keep = [
-        "packaging_format",
-        "id",
-        "name",
-        "description",
-        "version",
-        "maintainers",
-        "upstream",
-        "integration",
-        "install",
-        "resources",
-    ]
+    keys_to_keep = ["packaging_format", "id", "name", "description", "version", "maintainers", "upstream", "integration", "install", "resources"]

    keys_to_del = [key for key in manifest.keys() if key not in keys_to_keep]
    for key in keys_to_del:
--- a/tools/readme_generator/make_readme.py
+++ b/tools/readme_generator/make_readme.py
@ -59,9 +59,7 @@ def generate_READMEs(app_path: Path):
        if README_template.name == "README.md.j2":
            continue

-        if not README_template.name.endswith(
-            ".j2"
-        ) or not README_template.name.startswith("README_"):
+        if not README_template.name.endswith(".j2") or not README_template.name.startswith("README_"):
            continue

        language_code = README_template.name.split("_")[1].split(".")[0]
--- a/tools/readme_generator/tests/README.md
+++ b/tools/readme_generator/tests/README.md
@ -0,0 +1,60 @@
+<!--
+N.B.: This README was automatically generated by https://github.com/YunoHost/apps/tree/master/tools/readme_generator
+It shall NOT be edited by hand.
+-->
+
+# GoToSocial for YunoHost
+
+[![Integration level](https://dash.yunohost.org/integration/gotosocial.svg)](https://dash.yunohost.org/appci/app/gotosocial) ![Working status](https://ci-apps.yunohost.org/ci/badges/gotosocial.status.svg) ![Maintenance status](https://ci-apps.yunohost.org/ci/badges/gotosocial.maintain.svg)
+
+[![Install GoToSocial with YunoHost](https://install-app.yunohost.org/install-with-yunohost.svg)](https://install-app.yunohost.org/?app=gotosocial)
+
+*[Lire ce readme en français.](./README_fr.md)*
+
+> *This package allows you to install GoToSocial quickly and simply on a YunoHost server.
+If you don't have YunoHost, please consult [the guide](https://yunohost.org/#/install) to learn how to install it.*
+
+## Overview
+
+GoToSocial is a fast [ActivityPub](https://activitypub.rocks/) social network server, written in Golang.
+
+With GoToSocial, you can keep in touch with your friends, post, read, and share images and articles. All without being tracked or advertised to!
+
+The official documentation is at [docs.gotosocial.org](https://docs.gotosocial.org).  
+The documentation for this YunoHost package [can be read here](./doc/DOCS.md) and the admin is **strongly encouraged to read it**!
+
+Please note that this package uses the ["i'm so tired" software license 1.0](https://github.com/YunoHost-Apps/gotosocial_ynh/blob/master/LICENSE), please read it and accept it before proceeding with installation.
+
+**Shipped version:** 0.13.3~ynh1
+
+## Screenshots
+
+![Screenshot of GoToSocial](./doc/screenshots/screenshot.png)
+
+## :red_circle: Antifeatures
+
+- **Alpha software**: Early development stage. May contain changing or unstable features, bugs, and security vulnerability.
+- **Not totally free package**: The YunoHost package of this app is under an overall free licence, but with clauses that restrict its use.
+
+## Documentation and resources
+
+- Official app website: <https://gotosocial.org/>
+- Official user documentation: <https://docs.gotosocial.org/en/latest/>
+- Official admin documentation: <https://docs.gotosocial.org/en/latest/>
+- Upstream app code repository: <https://github.com/superseriousbusiness/gotosocial>
+- YunoHost Store: <https://apps.yunohost.org/app/gotosocial>
+- Report a bug: <https://github.com/YunoHost-Apps/gotosocial_ynh/issues>
+
+## Developer info
+
+Please send your pull request to the [testing branch](https://github.com/YunoHost-Apps/gotosocial_ynh/tree/testing).
+
+To try the testing branch, please proceed like that.
+
+``` bash
+sudo yunohost app install https://github.com/YunoHost-Apps/gotosocial_ynh/tree/testing --debug
+or
+sudo yunohost app upgrade gotosocial -u https://github.com/YunoHost-Apps/gotosocial_ynh/tree/testing --debug
+```
+
+**More info regarding app packaging:** <https://yunohost.org/packaging_apps>
--- a/tools/readme_generator/tests/README_fr.md
+++ b/tools/readme_generator/tests/README_fr.md
@ -0,0 +1,60 @@
+<!--
+N.B.: This README was automatically generated by https://github.com/YunoHost/apps/tree/master/tools/readme_generator
+It shall NOT be edited by hand.
+-->
+
+# GoToSocial pour YunoHost
+
+[![Niveau d’intégration](https://dash.yunohost.org/integration/gotosocial.svg)](https://dash.yunohost.org/appci/app/gotosocial) ![Statut du fonctionnement](https://ci-apps.yunohost.org/ci/badges/gotosocial.status.svg) ![Statut de maintenance](https://ci-apps.yunohost.org/ci/badges/gotosocial.maintain.svg)
+
+[![Installer GoToSocial avec YunoHost](https://install-app.yunohost.org/install-with-yunohost.svg)](https://install-app.yunohost.org/?app=gotosocial)
+
+*[Read this readme in english.](./README.md)*
+
+> *Ce package vous permet d’installer GoToSocial rapidement et simplement sur un serveur YunoHost.
+Si vous n’avez pas YunoHost, regardez [ici](https://yunohost.org/#/install) pour savoir comment l’installer et en profiter.*
+
+## Vue d’ensemble
+
+Un serveur de réseau social véloce basé sur [ActivityPub](https://activitypub.rocks/) écrit en Golang.
+
+Avec GoToSocial, vous pouvez rester en contact avec vos amis, publier, lire et partager des images et des articles. Tout cela sans être pisté ni subir de publicité !
+
+Vous pouvez consulter la documentation officielle à l'adresse : [docs.gotosocial.org](https://docs.gotosocial.org).  
+La documentation de ce paquet YunoHost [est lisible ici](./doc/DOCS_fr.md) et l'admin est **vivement encouragé-e à la lire** !
+
+Veuillez noter que ce paquet utilise la ["i'm so tired" software license 1.0](https://github.com/YunoHost-Apps/gotosocial_ynh/blob/master/LICENSE), veuillez la lire et l'accepter avant de procéder à l'installation.
+
+**Version incluse :** 0.13.3~ynh1
+
+## Captures d’écran
+
+![Capture d’écran de GoToSocial](./doc/screenshots/screenshot.png)
+
+## :red_circle: Anti-fonctionnalités
+
+- **Logiciel en version alpha** : Le logiciel est au tout début de son développement. Il pourrait contenir des fonctionnalités changeantes ou instables, des bugs, et des failles de sécurité.
+- **Package sous licence libre restreinte** : Le package YunoHost de cette application est sous une licence globalement libre, mais avec des clauses qui pourraient restreindre son utilisation.
+
+## Documentations et ressources
+
+- Site officiel de l’app : <https://gotosocial.org/>
+- Documentation officielle utilisateur : <https://docs.gotosocial.org/en/latest/>
+- Documentation officielle de l’admin : <https://docs.gotosocial.org/en/latest/>
+- Dépôt de code officiel de l’app : <https://github.com/superseriousbusiness/gotosocial>
+- YunoHost Store : <https://apps.yunohost.org/app/gotosocial>
+- Signaler un bug : <https://github.com/YunoHost-Apps/gotosocial_ynh/issues>
+
+## Informations pour les développeurs
+
+Merci de faire vos pull request sur la [branche testing](https://github.com/YunoHost-Apps/gotosocial_ynh/tree/testing).
+
+Pour essayer la branche testing, procédez comme suit.
+
+``` bash
+sudo yunohost app install https://github.com/YunoHost-Apps/gotosocial_ynh/tree/testing --debug
+ou
+sudo yunohost app upgrade gotosocial -u https://github.com/YunoHost-Apps/gotosocial_ynh/tree/testing --debug
+```
+
+**Plus d’infos sur le packaging d’applications :** <https://yunohost.org/packaging_apps>
--- a/tools/readme_generator/tests/tests_make_readme.py
+++ b/tools/readme_generator/tests/tests_make_readme.py
@ -0,0 +1,22 @@
+import os
+import subprocess
+
+CWD = os.path.split(os.path.realpath(__file__))[0]
+COMMIT_ID = "8f788213b363a46a5b6faa8f844d86d4adac9446"
+DIRECTORY = CWD + "/gotosocial_ynh"
+
+
+def test_running_make_readme():
+    subprocess.check_call(["rm", "-rf", DIRECTORY])
+    subprocess.check_call(["git", "clone", "https://github.com/yunohost-apps/gotosocial_ynh", DIRECTORY, "-q"])
+    subprocess.check_call(["git", "checkout", COMMIT_ID, "-q"], cwd=DIRECTORY)
+
+    print(CWD)
+    subprocess.check_call([CWD + "/../make_readme.py", DIRECTORY])
+
+    assert open(CWD + "/README.md").read() == open(DIRECTORY + "/README.md").read()
+    assert open(CWD + "/README_fr.md").read() == open(DIRECTORY + "/README_fr.md").read()
+
+
+if __name__ == '__main__':
+    test_running_make_readme()
--- a/tools/readme_generator/webhook.py
+++ b/tools/readme_generator/webhook.py
@ -35,21 +35,14 @@ async def git(cmd, in_folder=None):
    cmd = ["git"] + cmd
    cmd = " ".join(map(shlex.quote, cmd))
    print(cmd)
-    command = await asyncio.create_subprocess_shell(
-        cmd,
-        env=my_env,
-        stdout=asyncio.subprocess.PIPE,
-        stderr=asyncio.subprocess.STDOUT,
-    )
+    command = await asyncio.create_subprocess_shell(cmd, env=my_env, stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.STDOUT)
    data = await command.stdout.read()
    return data.decode().strip()


@app.route("/github", methods=["GET"])
 def main_route(request):
-    return text(
-        "You aren't supposed to go on this page using a browser, it's for webhooks push instead."
-    )
+    return text("You aren't supposed to go on this page using a browser, it's for webhooks push instead.")


@app.route("/github", methods=["POST"])
@ -65,9 +58,7 @@ async def on_push(request):
        return response.json({"error": "Signing algorightm is not sha1 ?!"}, 501)

    # HMAC requires the key to be bytes, but data is string
-    mac = hmac.new(
-        github_webhook_secret.encode(), msg=request.body, digestmod=hashlib.sha1
-    )
+    mac = hmac.new(github_webhook_secret.encode(), msg=request.body, digestmod=hashlib.sha1)

    if not hmac.compare_digest(str(mac.hexdigest()), str(signature)):
        return response.json({"error": "Bad signature ?!"}, 403)
@ -80,42 +71,19 @@ async def on_push(request):
    print(f"{repository} -> branch '{branch}'")

    with tempfile.TemporaryDirectory() as folder:
-        await git(
-            [
-                "clone",
-                f"https://{login}:{token}@github.com/{repository}",
-                "--single-branch",
-                "--branch",
-                branch,
-                folder,
-            ]
-        )
+        await git(["clone", f"https://{login}:{token}@github.com/{repository}", "--single-branch", "--branch", branch, folder])
        generate_READMEs(folder)

        await git(["add", "README*.md"], in_folder=folder)

-        diff_not_empty = await asyncio.create_subprocess_shell(
-            " ".join(["git", "diff", "HEAD", "--compact-summary"]),
-            cwd=folder,
-            stdout=asyncio.subprocess.PIPE,
-            stderr=asyncio.subprocess.STDOUT,
-        )
+        diff_not_empty = await asyncio.create_subprocess_shell(" ".join(["git", "diff", "HEAD", "--compact-summary"]), cwd=folder, stdout=asyncio.subprocess.PIPE, stderr=asyncio.subprocess.STDOUT)
        diff_not_empty = await diff_not_empty.stdout.read()
        diff_not_empty = diff_not_empty.decode().strip()
        if not diff_not_empty:
            print("nothing to do")
            return text("nothing to do")

-        await git(
-            [
-                "commit",
-                "-a",
-                "-m",
-                "Auto-update README",
-                "--author='yunohost-bot <yunohost@yunohost.org>'",
-            ],
-            in_folder=folder,
-        )
+        await git(["commit", "-a", "-m", "Auto-update README", "--author='yunohost-bot <yunohost@yunohost.org>'"], in_folder=folder)
        await git(["push", "origin", branch, "--quiet"], in_folder=folder)

    return text("ok")
--- a/tools/update_app_levels/update_app_levels.py
+++ b/tools/update_app_levels/update_app_levels.py
@ -107,8 +107,7 @@ def list_changes(catalog, ci_results) -> dict[str, list[tuple[str, int, int]]]:


 def pretty_changes(changes: dict[str, list[tuple[str, int, int]]]) -> str:
-    pr_body_template = textwrap.dedent(
-        """
+    pr_body_template = textwrap.dedent("""
        {%- if changes["major_regressions"] %}
        ### Major regressions 😭
        {% for app in changes["major_regressions"] %}
@ -139,8 +138,7 @@ def pretty_changes(changes: dict[str, list[tuple[str, int, int]]]) -> str:
        - [ ] [{{app}} (See latest job if it exists)](https://ci-apps.yunohost.org/ci/apps/{{app}}/latestjob)
        {%- endfor %}
        {% endif %}
-    """
-    )
+    """)

    return jinja2.Environment().from_string(pr_body_template).render(changes=changes)

@ -150,34 +148,24 @@ def make_pull_request(pr_body: str) -> None:
        "title": "Update app levels according to CI results",
        "body": pr_body,
        "head": "update_app_levels",
-        "base": "master",
+        "base": "master"
    }

    with requests.Session() as s:
        s.headers.update({"Authorization": f"token {github_token()}"})
-        response = s.post(
-            f"https://api.github.com/repos/{APPS_REPO}/pulls", json=pr_data
-        )
+        response = s.post(f"https://api.github.com/repos/{APPS_REPO}/pulls", json=pr_data)

        if response.status_code == 422:
-            response = s.get(
-                f"https://api.github.com/repos/{APPS_REPO}/pulls",
-                data={"head": "update_app_levels"},
-            )
+            response = s.get(f"https://api.github.com/repos/{APPS_REPO}/pulls", data={"head": "update_app_levels"})
            response.raise_for_status()
            pr_number = response.json()[0]["number"]

            # head can't be updated
            del pr_data["head"]
-            response = s.patch(
-                f"https://api.github.com/repos/{APPS_REPO}/pulls/{pr_number}",
-                json=pr_data,
-            )
+            response = s.patch(f"https://api.github.com/repos/{APPS_REPO}/pulls/{pr_number}", json=pr_data)
            response.raise_for_status()
            existing_url = response.json()["html_url"]
-            logging.warning(
-                f"An existing Pull Request has been updated at {existing_url} !"
-            )
+            logging.warning(f"An existing Pull Request has been updated at {existing_url} !")
        else:
            response.raise_for_status()