Skip to content

Fix CI lint job failures from Ruff rule violations - #22

Merged
johnny603 merged 2 commits into
mainfrom
copilot/fix-lint-job-failure
Sep 3, 2026
Merged

Fix CI lint job failures from Ruff rule violations#22
johnny603 merged 2 commits into
mainfrom
copilot/fix-lint-job-failure

Conversation

Copilot AI commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

The lint GitHub Actions job was failing due to Ruff violations introduced across multiple Python modules (primarily line-length, import-order/placement, and ambiguous-name findings). This PR normalizes those files to satisfy the repository’s enforced Ruff policy without changing runtime behavior.

  • Failure surface

    • Address Ruff violations reported in the failing run (E501, I001, E402, E741, plus one test-side unused local).
  • Code updates

    • Reflow long literals and command strings (notably puzzle metadata/validator strings) to stay within line-length limits.
    • Correct import ordering and move module imports to top-level where required.
    • Rename ambiguous locals (l-style identifiers) to explicit names for lint compliance.
  • Scope characteristics

    • Changes are mechanical/compliance-oriented across touched files; logic and interfaces remain unchanged.
# before
def key(l):
    return (
        l.get("id") in solved,
        difficulty_key(l.get("difficulty")),
    )

# after
def key(level):
    return (
        level.get("id") in solved,
        difficulty_key(level.get("difficulty")),
    )

Co-authored-by: johnny603 <23467609+johnny603@users.noreply.github.com>
@johnny603
johnny603 marked this pull request as ready for review September 3, 2026 06:22
Copilot AI changed the title [WIP] Fix the failing GitHub Actions job lint Fix CI lint job failures from Ruff rule violations Sep 3, 2026
@johnny603
johnny603 merged commit ba62756 into main Sep 3, 2026
6 of 8 checks passed
Copilot stopped work on behalf of johnny603 due to an error September 3, 2026 06:23
Copilot AI requested a review from johnny603 September 3, 2026 06:23
@johnny603
johnny603 deleted the copilot/fix-lint-job-failure branch September 3, 2026 06:23
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