Skip to content

ci: fix workflows after llama->ogx CLI rename#338

Merged
cdoern merged 1 commit intomainfrom
fix-ci-ogx-rename
Apr 30, 2026
Merged

ci: fix workflows after llama->ogx CLI rename#338
cdoern merged 1 commit intomainfrom
fix-ci-ogx-rename

Conversation

@cdoern
Copy link
Copy Markdown
Collaborator

@cdoern cdoern commented Apr 30, 2026

Summary

  • Update integration-tests.yml to invoke `ogx stack list-deps` (was `llama`); add `src/ogx_client/**` to path filter
  • Scope pre-commit ruff/blacken-docs/insert-license hooks to `src/(ogx_client|llama_stack_client)/lib/.*` so insert-license stops adding Meta headers to Stainless-generated files under `src/ogx_client/types/` and `src/ogx_client/_utils/`

Unblocks the failing CI on #337.

Test plan

  • Integration Tests workflow finds the `ogx` binary (was failing with `Failed to spawn: 'llama'`)
  • Pre-commit no longer rewrites generated SDK files

- integration-tests.yml: invoke ogx instead of llama; add src/ogx_client/** to path filter
- pre-commit: scope ruff/blacken-docs/insert-license to ogx_client/lib so insert-license stops adding Meta headers to Stainless-generated files under src/ogx_client/types and src/ogx_client/_utils
@cdoern cdoern merged commit e79ff48 into main Apr 30, 2026
56 checks passed
@cdoern cdoern deleted the fix-ci-ogx-rename branch April 30, 2026 15:41
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.

1 participant