Skip to content

Remove black from runtime dependencies, normalize with quality - #392

Closed
timkpaine wants to merge 3 commits into
togethercomputer:mainfrom
timkpaine:timkpaine-patch-2
Closed

timkpaine wants to merge 3 commits into
togethercomputer:mainfrom
timkpaine:timkpaine-patch-2

Conversation

@timkpaine

@timkpaine timkpaine commented Nov 4, 2025

Copy link
Copy Markdown

Describe your changes

Black is listed as a runtime dependency, but it doesn't look like it's used in the project at runtime.


Note

Move Black from runtime dependencies to the quality group and pin it to >=25.9.0,<26.0.

  • Dependencies:
    • Remove black from [tool.poetry.dependencies].
    • Pin black in [tool.poetry.group.quality.dependencies] to >=25.9.0,<26.0.

Written by Cursor Bugbot for commit 3360266. This will update automatically on new commits. Configure here.

@broly-code-security-scanner

Copy link
Copy Markdown

Broly Security Scan

Note

Clean scan
No vulnerabilities detected in this PR.

Note

Re-scan this PR anytime with /broly scan — useful after /broly undismiss, or to refresh findings without a new push.

Broly — SAST (zai-org/GLM-5.3-Flash) · Secrets · SCA · IaC · GH Actions · Base Images · Supply Chain Threats · Exploit Chains · Adversarial Verification

We're continuously improving Broly's accuracy and finding quality — your feedback is valuable. False positives, missed findings, bugs, and feature requests all welcome.

Ask in #security-engineering   Powered by Together AI

@blainekasten

Copy link
Copy Markdown
Contributor

This repo is deprecated and is not taking new patches. Thanks.

The new SDK is hosted at https://github.com/togethercomputer/together-py

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