docs(providers): align setup guidance - #434
Conversation
8cf2c56 to
b62550f
Compare
rng1995
left a comment
There was a problem hiding this comment.
[SkillSpector Review]
The added provider setup paths otherwise match their implementations, but the table would still advertise nv_build's retired default after merging into current main. Please update that model reference so the setup documentation is actually aligned. All required checks are green.
|
Updated the nv_build row to the current z-ai/glm-5.2 default requested in review. The latest CI run is green; ready for re-review. |
Signed-off-by: Deepak Jain <deepujain@gmail.com>
f57abef to
2ce0642
Compare
Signed-off-by: Deepak Jain <deepujain@gmail.com>
2ce0642 to
c053700
Compare
|
Re-review ping: nv_build default is z-ai/glm-5.2 on current head c053700; all checks green. |
|
Re-review request on head The Aug 28 nv_build default comment is addressed: provider setup docs now reference CI green on this head. Prior review thread is outdated. Please re-review when convenient. |
Summary
.env.example, the README provider reference, and the development setup guide with the provider selector on currentmain.Validation
src/skillspector/providers/..env.example,README.md, anddocs/DEVELOPMENT.md.pytest -m 'not integration and not provider' tests/ -q— 2,825 passed, 13 skipped, 38 deselected, 4 xfailed.ruff check src tests— passed.ruff format --check src tests— 193 files already formatted.git diff --check— passed.Risk
Fixes #433