marvin8/cang
Watch
1
0
Fork
You've already forked cang
 0
Code Issues 1 Pull requests Releases 28 Activity Actions

Fix agent-reviews pipeline: all reviews via DeepSeek, lowercase secrets #88

New issue
Closed
opened 2026-06-05 06:40:50 +00:00 by coding-agent-marvin8 · 1 comment
coding-agent-marvin8 commented 2026-06-05 06:40:50 +00:00 (Migrated from codeberg.org)
Copy link

The new agent-reviews CI pipeline and script have several issues to address before they can be merged:

  1. YAML: from_secret: ANTHROPIC_API_KEY — remove entirely (switching to DeepSeek)
  2. YAML: from_secret: DEEPSEEK_API_KEY — must be lowercase
  3. YAML: from_secret: CODEBERG_TOKEN — must be lowercase
  4. Python: missing SPDX-License-Identifier header
  5. Python: ANTHROPIC_API_KEY config var and call_anthropic() function — remove both
  6. Python: Security agent uses Anthropic backend — switch to DeepSeek
  7. Python: Performance agent uses Anthropic backend — switch to DeepSeek
  8. Python: resp variable name violates no-abbreviation rule
  9. Python: raw variable name violates no-abbreviation rule
  10. Python: c loop variable in post_review() violates no-abbreviation rule
The new agent-reviews CI pipeline and script have several issues to address before they can be merged: 1. YAML: `from_secret: ANTHROPIC_API_KEY` — remove entirely (switching to DeepSeek) 2. YAML: `from_secret: DEEPSEEK_API_KEY` — must be lowercase 3. YAML: `from_secret: CODEBERG_TOKEN` — must be lowercase 4. Python: missing SPDX-License-Identifier header 5. Python: `ANTHROPIC_API_KEY` config var and `call_anthropic()` function — remove both 6. Python: Security agent uses Anthropic backend — switch to DeepSeek 7. Python: Performance agent uses Anthropic backend — switch to DeepSeek 8. Python: `resp` variable name violates no-abbreviation rule 9. Python: `raw` variable name violates no-abbreviation rule 10. Python: `c` loop variable in `post_review()` violates no-abbreviation rule
coding-agent-marvin8 commented 2026-06-05 06:41:25 +00:00 (Migrated from codeberg.org)
Copy link

Two-commit change. First commit: lowercase secrets + remove ANTHROPIC_API_KEY from YAML. Second commit: SPDX header, switch Security and Performance agents to deepseek-chat, remove call_anthropic(), fix naming (resp, raw, c).

Two-commit change. First commit: lowercase secrets + remove ANTHROPIC_API_KEY from YAML. Second commit: SPDX header, switch Security and Performance agents to deepseek-chat, remove call_anthropic(), fix naming (resp, raw, c).
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
pages
2026.8.4
2026.7.25
2026.7.24
2026.7.23
2026.7.22
0.10.1
0.10.0
0.9.0
0.8.0
0.7.0
0.6.0
0.5.0
0.4.2
0.4.1
0.4.0
0.3.12
0.3.11
0.3.10
0.3.9
0.3.8
0.3.7
0.3.6
0.3.5
0.3.4
0.3.3
0.3.2
0.3.1
0.3.0
0.2.0
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
marvin8/cang#88
No description provided.