Skip to content

Fix python-checks failure by normalizing Ruff formatting in model install test#9267

Closed
lstein with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-python-checks-job
Closed

Fix python-checks failure by normalizing Ruff formatting in model install test#9267
lstein with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-python-checks-job

Conversation

Copilot AI commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

python-checks failed in CI because ruff format --check reported tests/app/services/model_install/test_model_install.py as not formatted. This PR applies the minimal formatting-only correction so the workflow passes consistently.

  • Root cause

    • ruff@0.11.2 format --check required one additional blank line between top-level test functions in tests/app/services/model_install/test_model_install.py.
  • Change made

    • Applied Ruff-compliant whitespace normalization in the affected test file.
    • No behavior or test logic changes.
        release_restore.set()
        installer.stop()


def test_huggingface_blob_url_uses_resolve_download_url(...):

Related Issues / Discussions

N/A

QA Instructions

Run the same CI command locally:

uv tool run ruff@0.11.2 format --check .

Merge Plan

Normal merge; no special sequencing required.

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

Copilot AI changed the title [WIP] Fix failing GitHub Actions job python-checks Fix python-checks failure by normalizing Ruff formatting in model install test Jun 4, 2026
Copilot AI requested a review from lstein June 4, 2026 23:55
Copilot finished work on behalf of lstein June 4, 2026 23:55
@lstein lstein closed this Jun 5, 2026
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