Skip to content

Bug: PR Safety workflow failing on main — ruff format check fails for tests/test_docs_pipeline.py #236

Description

@Fumer057

Summary

The PR Safety workflow ("Lint and format" job) is failing on main because tests/test_docs_pipeline.py does not pass ruff format --check.

Failure Details

  • Workflow: .github/workflows/tests.yml → "PR Safety"
  • Job: Lint and format
  • Step: Check formatting
  • Failing run: PR Safety #49
  • Command: ruff format --check docs-agent-mcp/mcp-server tests

Error output:

Would reformat: tests/test_docs_pipeline.py
1 file would be reformatted, 15 files already formatted
Error: Process completed with exit code 1.

Impact

Every new PR against main inherits this formatting failure, which blocks contributors from getting clean CI results in the "Lint and format" job.

Root Cause

tests/test_docs_pipeline.py has a trailing blank line at the end of the file that ruff format flags as a formatting violation.

Fix

Running ruff format tests/test_docs_pipeline.py resolves the issue. PR: #235

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions