Force single quotes with ruff

Author: hugovk

.pre-commit-config.yaml

@@ -3,19 +3,16 @@ repos:
     rev: v0.8.4
     hooks:
       - id: ruff
-        args: [--exit-non-zero-on-fix]
-
-  - repo: https://github.com/psf/black-pre-commit-mirror
-    rev: 24.10.0
-    hooks:
-      - id: black
+        args: [--fix, --exit-non-zero-on-fix]
+      - id: ruff-format
 
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v5.0.0
     hooks:
       - id: check-added-large-files
       - id: check-case-conflict
       - id: check-merge-conflict
+      - id: check-toml
       - id: check-yaml
       - id: debug-statements
       - id: end-of-file-fixer

completion.py

@@ -11,22 +11,22 @@
 
 @cache
 def branches_from_devguide(devguide_dir: Path) -> list[str]:
-    p = devguide_dir.joinpath("include/release-cycle.json")
+    p = devguide_dir.joinpath('include/release-cycle.json')
     data = json.loads(p.read_text())
     return [
-        branch for branch in data if data[branch]["status"] in ("bugfix", "security")
+        branch for branch in data if data[branch]['status'] in ('bugfix', 'security')
     ]
 
 
 def get_completion(clones_dir: str, repo: str) -> tuple[float, int]:
     clone_path = Path(clones_dir, repo)
-    for branch in branches_from_devguide(Path(clones_dir, "devguide")) + ["master"]:
+    for branch in branches_from_devguide(Path(clones_dir, 'devguide')) + ['master']:
         try:
             git.Repo.clone_from(
-                f"https://github.com/{repo}.git", clone_path, branch=branch
+                f'https://github.com/{repo}.git', clone_path, branch=branch
             )
         except git.GitCommandError:
-            print(f"failed to clone {repo} {branch}")
+            print(f'failed to clone {repo} {branch}')
             translators_number = 0
             continue
         else:
@@ -35,9 +35,9 @@ def get_completion(clones_dir: str, repo: str) -> tuple[float, int]:
     with TemporaryDirectory() as tmpdir:
         completion = potodo.merge_and_scan_path(
             clone_path,
-            pot_path=Path(clones_dir, "cpython/Doc/build/gettext"),
+            pot_path=Path(clones_dir, 'cpython/Doc/build/gettext'),
             merge_path=Path(tmpdir),
             hide_reserved=False,
-            api_url="",
+            api_url='',
         ).completion
     return completion, translators_number

generate.py

@@ -26,20 +26,20 @@
 
 with TemporaryDirectory() as clones_dir:
     Repo.clone_from(
-        "https://github.com/python/devguide.git",
-        devguide_dir := Path(clones_dir, "devguide"),
+        'https://github.com/python/devguide.git',
+        devguide_dir := Path(clones_dir, 'devguide'),
         depth=1,
     )
     latest_branch = branches_from_devguide(devguide_dir)[0]
     Repo.clone_from(
-        "https://github.com/python/cpython.git",
-        Path(clones_dir, "cpython"),
+        'https://github.com/python/cpython.git',
+        Path(clones_dir, 'cpython'),
         depth=1,
         branch=latest_branch,
     )
-    subprocess.run(["make", "-C", Path(clones_dir, "cpython/Doc"), "venv"], check=True)
+    subprocess.run(['make', '-C', Path(clones_dir, 'cpython/Doc'), 'venv'], check=True)
     subprocess.run(
-        ["make", "-C", Path(clones_dir, "cpython/Doc"), "gettext"], check=True
+        ['make', '-C', Path(clones_dir, 'cpython/Doc'), 'gettext'], check=True
     )
     switcher_languages = list(switcher.get_languages())
     for language, repo in repositories.get_languages_and_repos(devguide_dir):
@@ -124,5 +124,5 @@
     completion_progress=completion_progress, generation_time=generation_time
 )
 
-with open("index.html", "w") as file:
+with open('index.html', 'w') as file:
     file.write(output)

repositories.py

@@ -9,15 +9,15 @@
 def get_languages_and_repos(
     devguide_dir: pathlib.Path,
 ) -> Generator[tuple[str, Optional[str]], None, None]:
-    translating = devguide_dir.joinpath("documentation/translating.rst").read_text()
+    translating = devguide_dir.joinpath('documentation/translating.rst').read_text()
     doctree = core.publish_doctree(translating)
 
     for node in doctree.traverse(table):
         for row_node in node.traverse(row)[1:]:
             language = row_node[0].astext()
             repo = row_node[2].astext()
             language_code = (
-                re.match(r".* \((.*)\)", language).group(1).lower().replace("_", "-")
+                re.match(r'.* \((.*)\)', language).group(1).lower().replace('_', '-')
             )
-            repo_match = re.match(":github:`GitHub <(.*)>`", repo)
+            repo_match = re.match(':github:`GitHub <(.*)>`', repo)
             yield language_code, repo_match and repo_match.group(1)

ruff.toml

@@ -0,0 +1,2 @@
+[format]
+quote-style = "single"

switcher.py

@@ -13,24 +13,24 @@
 
 def get_languages() -> Generator[str, None, None]:
     data = requests.get(
-        "https://raw.githubusercontent.com/"
-        "python/docsbuild-scripts/refs/heads/main/config.toml",
+        'https://raw.githubusercontent.com/'
+        'python/docsbuild-scripts/refs/heads/main/config.toml',
         timeout=10,
     ).text
     config = tomllib.loads(data)
-    languages = config["languages"]
-    defaults = config["defaults"]
+    languages = config['languages']
+    defaults = config['defaults']
     for code, language in languages.items():
-        if language.get("in_prod", defaults["in_prod"]):
-            yield code.lower().replace("_", "-")
+        if language.get('in_prod', defaults['in_prod']):
+            yield code.lower().replace('_', '-')
 
 
 def main() -> None:
     languages = list(get_languages())
     print(languages)
-    for code in ("en", "pl", "ar", "zh-cn"):
-        print(f"{code}: {code in languages}")
+    for code in ('en', 'pl', 'ar', 'zh-cn'):
+        print(f'{code}: {code in languages}')
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     main()

translators.py

@@ -12,20 +12,20 @@ def get_number(path: Path) -> int:
 
 
 def get_number_from_git_history(path: Path) -> int:
-    return len(Repo(path).git.shortlog("-s", "HEAD").splitlines())
+    return len(Repo(path).git.shortlog('-s', 'HEAD').splitlines())
 
 
 def yield_from_headers(path: Path) -> Generator[str, None, None]:
-    for file in path.rglob("*.po"):
+    for file in path.rglob('*.po'):
         try:
             header = pofile(file).header.splitlines()
         except IOError:
             continue
-        if "Translators:" not in header:
+        if 'Translators:' not in header:
             continue
-        for translator_record in header[header.index("Translators:") + 1 :]:
+        for translator_record in header[header.index('Translators:') + 1 :]:
             try:
-                translator, _year = translator_record.split(", ")
+                translator, _year = translator_record.split(', ')
             except ValueError:
                 yield translator_record
             else:

visitors.py

@@ -8,11 +8,12 @@
 
 def get_number_of_visitors(language: str) -> int:
     param = urllib.parse.urlencode(
-        {"filters": f'[["contains","event:page",["/{language}/"]]]'}
+        {'filters': f'[["contains","event:page",["/{language}/"]]]'}
     )
-    r = requests.get(f"https://plausible.io/docs.python.org/export?{param}", timeout=10)
-    with zipfile.ZipFile(io.BytesIO(r.content), "r") as z, z.open(
-        "visitors.csv"
-    ) as csv_file:
+    r = requests.get(f'https://plausible.io/docs.python.org/export?{param}', timeout=10)
+    with (
+        zipfile.ZipFile(io.BytesIO(r.content), 'r') as z,
+        z.open('visitors.csv') as csv_file,
+    ):
         csv_reader = csv.DictReader(io.TextIOWrapper(csv_file))
-        return sum(int(row["visitors"]) for row in csv_reader)
+        return sum(int(row['visitors']) for row in csv_reader)