Skip to content

Fix backend (py3.11) CI failure by applying Ruff formatting to precompute module - #9

Draft
pinthoz with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-backend-py3-11-job
Draft

pinthoz with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-backend-py3-11-job

Conversation

Copilot AI commented Sep 9, 2026

Copy link
Copy Markdown

What

backend (py3.11) failed in CI because ruff format --check src tests detected formatting drift in src/footyvision/ml/precompute.py. This PR applies the exact formatter-aligned edits required so the job passes without changing runtime behavior.

  • Formatting-only correction in precompute.py
    • Reflowed the shortlist list-comprehension slice into Ruff’s preferred wrapped form.
    • Reflowed the models dict-comprehension into Ruff’s preferred multi-line block.
"models": {
    name: _model_meta(model)
    for name, model in zip(TARGETS, (group, role, exact), strict=True)
}

Why

The failing check run (backend (py3.11)) was caused by style non-compliance, not functional defects. Aligning this file with enforced Ruff formatting removes the CI blocker with minimal, non-behavioral change.

Checklist

  • ruff check src tests and ruff format --check src tests pass
  • pytest passes (tests stay DB-, network- and LLM-free)
  • Docs updated (README / docs/) if behaviour or setup changed
  • No credentials, .env files or datasets committed

@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
footy-vision Ready Ready Preview Sep 9, 2026 1:25pm UTC

Co-authored-by: pinthoz <69254873+pinthoz@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job backend (py3.11) Fix backend (py3.11) CI failure by applying Ruff formatting to precompute module Sep 9, 2026
Copilot AI requested a review from pinthoz September 9, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants