Skip to content

Fix badge workflow PR comment permissions#12

Merged
bsbodden merged 1 commit into
mainfrom
fix-badge-pr-comment-permissions
May 16, 2026
Merged

Fix badge workflow PR comment permissions#12
bsbodden merged 1 commit into
mainfrom
fix-badge-pr-comment-permissions

Conversation

@bsbodden

Copy link
Copy Markdown
Member

Summary

  • Add PR comment permissions to the badge workflow token
  • Treat comment-posting failures as warnings so badge calculation remains the required signal

Investigation

The failing badge action calculates the MFCQI score successfully, then fails while posting the PR comment with 403 Resource not accessible by integration. The job token only had contents: write, while the API response requires issues=write or pull_requests=write.

Validation

  • Parsed .github/workflows/mfcqi-badge.yml with PyYAML

@github-actions

Copy link
Copy Markdown
Contributor

✅ MFCQI Score: 0.87 → (±0.000)

➡️ Previous: 0.87

Code quality meets the minimum threshold (0.70).

@bsbodden bsbodden self-assigned this May 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Code Quality Report

Metric Score Status
MFCQI Score 0.8713324632676854 ✅ Good
Test Coverage 82% ✅ Good
Security Issues 0 ✅ Clean

Quality Gates

  • MFCQI Score: ✅ Minimum 0.75 (Current: 0.8713324632676854)
  • Test Coverage: ✅ Minimum 80% (Current: 82%)
  • Security: ✅ Zero vulnerabilities (Current: 0)

🎉 All quality gates passed! This PR maintains high code quality standards.

📊 View detailed MFCQI breakdown

Run mfcqi analyze src/mfcqi locally to see detailed metrics breakdown.

@bsbodden bsbodden merged commit d2465d7 into main May 16, 2026
15 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