Skip to content

chore: add pull covergae step to itself#3

Merged
CodeYogiCo merged 5 commits into
masterfrom
Self-add
May 21, 2026
Merged

chore: add pull covergae step to itself#3
CodeYogiCo merged 5 commits into
masterfrom
Self-add

Conversation

@CodeYogiCo
Copy link
Copy Markdown
Owner

❤ THANKS FOR YOUR PULL REQUEST

Proposed change

Describe your change and communicate why we should accept the pull request. Links to issues are helpful too, if applicable.

Types of changes

What types of changes is this pull request introducing to this plugin? Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

You can fill this out after creating your PR. Put an x in the boxes that apply

  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

CodeYogiCo and others added 2 commits May 21, 2026 14:46
Touches an executed line in Run() so the dogfooded pr-code-coverage
workflow has a covered Go change to report on.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Code Coverage Summary:

Lines Without Coverage Data -> 0% (0)
Lines With Coverage Data -> 100% (2)
Covered Instructions -> 100% (2)
Missed Instructions -> 0% (0)

CodeYogiCo and others added 3 commits May 21, 2026 15:09
Refactor the PR comment into friendlier GitHub markdown: an H2 heading,
a status emoji driven by covered %, a table mapping each metric to a
plain-language explanation, and a collapsible missed-instructions block.
Console (Simple) reporter output is unchanged.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
The master merge kept the old /api/v3/repos assertions while the merged
github_pr.go posts to /repos; update the six body-test URLs to match.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

📊 Code Coverage — Changed Lines

Coverage is measured only for the lines this PR changes, not the whole file or repo.

🟢 Covered Instructions: 96% (50)

Metric Result What it means
🟢 Covered Instructions 96% (50) Changed code your tests executed. Higher is better.
🔴 Missed Instructions 4% (2) Changed code your tests never ran. Lower is better.
📈 Lines With Coverage Data 31% (52) Changed lines the coverage tool could track.
⚪ Lines Without Coverage Data 69% (118) Changed lines with no data: comments, blanks, declarations.
🔍 Missed instructions (2)
--- internal/plugin/reporter/github_pr.go:171
	default:
--- internal/plugin/reporter/github_pr.go:172
		return "🔴"

🤖 Generated by pull-request-code-coverage — coverage for changed lines only.

@CodeYogiCo CodeYogiCo merged commit 5dc1fa9 into master May 21, 2026
2 checks passed
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