Fix coverage badge: parse Cover column instead of Miss#29
Fix coverage badge: parse Cover column instead of Miss#29TheRealAgentK merged 19 commits intomasterfrom
Conversation
… validation, JSON parse fallback
Coverage —
|
| File | Stmts | Miss | Cover | Missing |
|---|---|---|---|---|
src/autohive_integrations_sdk/__init__.py |
2 | 0 | 100% | |
src/autohive_integrations_sdk/integration.py |
358 | 3 | 99% | 308, 315, 347 |
💡 Codex Reviewintegrations-sdk/.github/workflows/tests.yml Lines 5 to 7 in 463238a This workflow is scoped to ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
The coverage badge was showing
3(the Miss count) instead of99%because the awk parser used$(NF-1)which grabbed column 3 instead of column 4.Fix:
awk '{print $4}'explicitly targets the Cover column from the TOTAL line.