feat: bump to python 3.14, lint/fmt in ci (#22)

Author: aidandenlinger

.github/workflows/lint-fmt-py.yml

@@ -0,0 +1,37 @@
+name: Lint/Format with Ruff
+
+on:
+  push:
+    branches:
+      - main
+    paths:
+      - "**/*.py"
+  pull_request:
+    paths:
+      - "**/*.py"
+  workflow_dispatch:
+
+permissions: {}
+
+jobs:
+  lint-fmt-py:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+        with:
+          persist-credentials: false
+
+      # Runs `ruff check`
+      - uses: astral-sh/ruff-action@57714a7c8a2e59f32539362ba31877a1957dded1 # v3.5.1
+
+      # Check formatting
+      - run: ruff format --check --diff
+
+      - uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
+        with:
+          python-version: '3.14'
+
+      - run: "python -m pip install types-requests mypy"
+
+      - run: "mypy ."

.github/workflows/lint-fmt-sh.yml

@@ -0,0 +1,26 @@
+name: Lint/Format with Shellcheck
+
+on:
+  push:
+    branches:
+      - main
+    paths:
+      - "**/*.sh"
+  pull_request:
+    paths:
+      - "**/*.sh"
+  workflow_dispatch:
+
+permissions: {}
+
+jobs:
+  lint-fmt-sh:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+        with:
+          persist-credentials: false
+
+      - name: Run ShellCheck
+        run: find . -type f -name "*.sh" -exec shellcheck {} +

COPR.md

@@ -39,7 +39,7 @@ To set up autobuilds:
 
 - Select individual events, select branch/tag creation, and deselect push.
   - We don't want to fire the webhook on every commit, because some commits
-    won't change this specific package! autocopr.py attaches the cooresponding
+    won't change this specific package! The script attaches the cooresponding
     tags when needed.
 
 From that point, it just works! Whenever the Github Action runs, if there's an

README.md

@@ -190,10 +190,10 @@ jobs:
 ### As a Standalone Python Script
 
 ```shell
-python autocopr.py <folder-with-spec-files>
+python main.py <folder-with-spec-files>
 ```
 
-will run the `autocopr.py` script, searching for spec files in the provided directory.
+will run the `main.py` script, searching for spec files in the provided directory.
 If you do not provide a folder, it will search the current directory.
 
 By default, running the script will

action.sh

@@ -28,4 +28,4 @@ fi
 # https://stackoverflow.com/a/11114547
 ACTIONS_FOLDER=$(dirname "$(realpath --no-symlinks "${BASH_SOURCE[0]}")")
 
-python "${ACTIONS_FOLDER}"/autocopr.py "${args[@]}" "${ROOT_LOC}"
+python "${ACTIONS_FOLDER}"/main.py "${args[@]}" "${ROOT_LOC}"

action.yml

@@ -27,7 +27,7 @@ runs:
   steps:
     - uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
       with:
-        python-version: '3.13'
+        python-version: '3.14'
         # FIXME: Caching outside of the directory is not supported right now.
         # I could work around it, but frankly I'm only installing requests, it isn't worth it
         # Implement when the issue is resolved

autocopr/update.py

@@ -52,7 +52,9 @@ def update_version(
         exit(1)
 
     commit_msg = f"Update {spec.name} to {latest.ver}\n\n{latest.url}"
-    commit_result = subprocess.run(["git", "commit", "-m", commit_msg], capture_output=(not verbose))
+    commit_result = subprocess.run(
+        ["git", "commit", "-m", commit_msg], capture_output=(not verbose)
+    )
     if commit_result.returncode:
         logging.error(
             "Failed to make a commit.\nThis is a bug, please report it. Exiting..."
@@ -76,7 +78,9 @@ def update_version(
 
     # The github webhooks won't fire if 3+ tags are made at once, to be
     # defensive we push on each spec file rather than pushing at the end
-    push_result = subprocess.run(["git", "push", "--follow-tags"], capture_output=(not verbose))
+    push_result = subprocess.run(
+        ["git", "push", "--follow-tags"], capture_output=(not verbose)
+    )
     if push_result.returncode:
         if not verbose:
             logging.error(add_result.stderr)

dev-requirements.txt

@@ -140,7 +140,7 @@ def get_latest_versions(
     if "data" not in resp or "nodes" not in resp["data"]:
         logging.error(f"GraphQL response not in expected shape: {resp}")
         exit(1)
-        
+
     for spec, node in zip((spec for (spec, _) in spec_ids), resp["data"]["nodes"]):
         if node and (latest := node["latestRelease"]) and "tagName" in latest:
             latest_version = clean_tag(latest["tagName"])

githubapi/graphql.py

@@ -63,7 +63,9 @@ def main():
 
     print("\n".join(update_summary))
 
-    had_updates = [(spec, latest) for (spec, latest) in latest_vers if spec.version != latest.ver]
+    had_updates = [
+        (spec, latest) for (spec, latest) in latest_vers if spec.version != latest.ver
+    ]
 
     if len(had_updates) == 0:
         print("All spec files are up to date!")
@@ -81,10 +83,14 @@ def main():
                         "All spec files are now up to date. If updated, the original spec file is backed up as a .bak file."
                     )
                 else:
-                    print("All spec files are now up to date, and the updates have been pushed.")
-        
+                    print(
+                        "All spec files are now up to date, and the updates have been pushed."
+                    )
+
         case Mode.DryRun | Mode.Check:
-            print(f"{[spec.name for (spec, _) in had_updates]} {"is" if len(had_updates) == 1 else "are"} outdated.")
+            print(
+                f"{[spec.name for (spec, _) in had_updates]} {'is' if len(had_updates) == 1 else 'are'} outdated."
+            )
             if args.mode == Mode.Check:
                 print("Exiting with an error because we are in check mode.")
                 exit(1)