Skip to content

feat: enhance CI setup with model-specific API key handling and comme…#1

Merged
starc007 merged 2 commits intomainfrom
dev
Mar 17, 2026
Merged

feat: enhance CI setup with model-specific API key handling and comme…#1
starc007 merged 2 commits intomainfrom
dev

Conversation

@starc007
Copy link
Owner

…nt acknowledgment

@github-actions
Copy link

github-actions bot commented Mar 17, 2026

crev review

🔴 HIGH src/main.rs:642 — The environment variable key in the CI workflow is set to __SECRET__ instead of the actual secret name (e.g., OPENAI_API_KEY), causing the API call to fail due to a missing/unresolved secret.
🟡 MED src/main.rs:646 — The run crev review step in the CI workflow uses __MODEL__ as a literal string instead of the dynamically injected model name, preventing the model configuration from taking effect.

1 high, 1 med · powered by crev

@starc007 starc007 merged commit 87903ee into main Mar 17, 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.

1 participant