Skip to content

Exclude vendored PDF tools from Ruff linting#1

Merged
c-rosenberg merged 2 commits into
mainfrom
copilot/fix-lint-job
Jul 9, 2026
Merged

Exclude vendored PDF tools from Ruff linting#1
c-rosenberg merged 2 commits into
mainfrom
copilot/fix-lint-job

Conversation

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

The GitHub Actions Lint job was failing because Ruff was analyzing vendored third-party PDF tooling under src/xspct_scan/vendor/. This change scopes Ruff to repository-owned code so the lint job no longer fails on upstream style violations.

  • Lint scope

    • Add a Ruff exclusion for the vendored PDF tools directory in pyproject.toml.
    • Leave the vendored pdfid.py / pdf_parser.py sources unchanged.
  • Why this change

    • The failing violations came from third-party code imported as vendor assets, not from maintained project code.
    • Excluding that directory keeps CI aligned with the codebase this repository owns.
[tool.ruff]
extend-exclude = ["src/xspct_scan/vendor"]

Copilot AI changed the title [WIP] Fix failing GitHub Actions job "Lint" Exclude vendored PDF tools from Ruff linting Jul 9, 2026
Copilot AI requested a review from c-rosenberg July 9, 2026 09:17
@c-rosenberg
c-rosenberg marked this pull request as ready for review July 9, 2026 09:34
@c-rosenberg
c-rosenberg merged commit a9a53a8 into main Jul 9, 2026
7 of 8 checks passed
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