You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fixed in PR #54 (commit 36200e0): requests_toolbelt dependency was added to pyproject.toml in v0.6.0 (SSRF DNS-rebinding fix, PR #52) but nightly.yml's test-latest-deps and test-min-deps jobs install ForgeLM with pip install -e . --no-deps after a manual pip install of the dependency list — so the manual list MUST mirror pyproject.toml dependencies, and requests-toolbelt was missing.
Added to both job's install lines (test-latest-deps unconstrained; test-min-deps pinned to 1.0.0 as the documented lower bound) + a comment block spelling out the mirror contract so a future dependency addition can't recreate the gap. Verified by re-running the nightly workflow successfully. Closing.
Activity
cemililik commentedon May 15, 2026
Fixed in PR #54 (commit 36200e0):
requests_toolbeltdependency was added topyproject.tomlin v0.6.0 (SSRF DNS-rebinding fix, PR #52) butnightly.yml'stest-latest-depsandtest-min-depsjobs install ForgeLM withpip install -e . --no-depsafter a manual pip install of the dependency list — so the manual list MUST mirrorpyproject.tomldependencies, andrequests-toolbeltwas missing.Added to both job's install lines (
test-latest-depsunconstrained;test-min-depspinned to1.0.0as the documented lower bound) + a comment block spelling out the mirror contract so a future dependency addition can't recreate the gap. Verified by re-running the nightly workflow successfully. Closing.