Skip to content

Fix coverage badge: parse Cover column instead of Miss#29

Merged
TheRealAgentK merged 19 commits intomasterfrom
feature/test-setup
Apr 10, 2026
Merged

Fix coverage badge: parse Cover column instead of Miss#29
TheRealAgentK merged 19 commits intomasterfrom
feature/test-setup

Conversation

@TheRealAgentK
Copy link
Copy Markdown
Collaborator

The coverage badge was showing 3 (the Miss count) instead of 99% 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.

@TheRealAgentK TheRealAgentK requested a review from ProRedCat April 10, 2026 01:14
@github-actions
Copy link
Copy Markdown

Coverage — d733de7 (Merge branch 'master' into feature/test-setup) by @TheRealAgentK

Total coverage: 99%

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

@TheRealAgentK TheRealAgentK merged commit aa92483 into master Apr 10, 2026
1 check passed
@TheRealAgentK TheRealAgentK deleted the feature/test-setup branch April 10, 2026 01:16
@chatgpt-codex-connector
Copy link
Copy Markdown

💡 Codex Review

branches: [master]
pull_request:
branches: [master]

P1 Badge Target the active branch in workflow triggers

This workflow is scoped to branches: [master], which means pushes and PRs against main do not run the test job at all; in this repository the active branch is main, so CI and coverage reporting are effectively skipped for normal development. Please switch the branch filters (and the later refs/heads/master badge condition) to main or include both branch names.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

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.

2 participants