Skip to content

Stabilize setup.py formatting for the pre-commit workflow - #23

Closed
sw005320 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-pre-commit-job
Closed

Stabilize setup.py formatting for the pre-commit workflow#23
sw005320 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-pre-commit-job

Conversation

Copilot AI commented Aug 10, 2026

Copy link
Copy Markdown

The pre-commit GitHub Actions job was failing because black reformatted setup.py in CI. This change makes the affected dependency list expression structurally explicit so it remains stable under Black.

  • CI failure source

    • Normalize the install_requires expression in setup.py, which was the only file reported by the failing black hook.
  • Change

    • Wrap the requirements + [...] expression in explicit parentheses inside setup(...).
    • Preserve existing packaging behavior; this is a formatting-stability change only.
  • Resulting shape

    install_requires=(
        requirements
        + [
            str(r)
            for r in pkg_resources.parse_requirements(
                open(os.path.join(os.path.dirname(__file__), "requirements.txt"))
            )
        ]
    ),

Co-authored-by: sw005320 <11741550+sw005320@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job pre-commit Stabilize setup.py formatting for the pre-commit workflow Aug 10, 2026
Copilot AI requested a review from sw005320 August 10, 2026 11:58
@sw005320 sw005320 closed this Aug 10, 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