Skip to content

Fix ruff lint failures across pkg/ and tests/#2

Merged
art-test-stack merged 3 commits into
copilot-vibe-codefrom
copilot/sub-pr-1
Mar 17, 2026
Merged

Fix ruff lint failures across pkg/ and tests/#2
art-test-stack merged 3 commits into
copilot-vibe-codefrom
copilot/sub-pr-1

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 17, 2026

CI was failing on ruff check --select=E,W,F,I and ruff format --check due to accumulated lint issues in source and test files.

Changes

  • Import hygiene – sorted import blocks (I001), removed unused imports sys and MagicMock (F401)
  • Whitespace – stripped trailing whitespace from blank lines (W293)
  • Line length – broke lines exceeding 88 chars (E501) in cli.py, graph.py, metadata.py, and test files; examples:
    # Before
    title = work.get("title", [""])[0] if isinstance(work.get("title"), list) else work.get("title", "Unknown")
    
    # After
    title = (
        work.get("title", [""])[0]
        if isinstance(work.get("title"), list)
        else work.get("title", "Unknown")
    )
  • Dead code – removed unused cv_section_start assignment and a bare result.rfind(...) call whose return value was discarded in test_markdown.py
  • Formatting – applied ruff format to all 15 files

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI mentioned this pull request Mar 17, 2026
@art-test-stack art-test-stack marked this pull request as ready for review March 17, 2026 11:06
Copilot AI and others added 2 commits March 17, 2026 11:09
… unused vars, I001 import order

Co-authored-by: art-test-stack <110672812+art-test-stack@users.noreply.github.com>
Co-authored-by: art-test-stack <110672812+art-test-stack@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix lint failures in code changes Fix ruff lint failures across pkg/ and tests/ Mar 17, 2026
Copilot AI requested a review from art-test-stack March 17, 2026 11:12
Copy link
Copy Markdown
Owner

@art-test-stack art-test-stack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@art-test-stack art-test-stack merged commit ae42fcc into copilot-vibe-code Mar 17, 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