Skip to content

Fix proposal approval status automation - #162

Merged
ewels merged 1 commit into
mainfrom
fix/proposal-approval-automation
Aug 14, 2026
Merged

Fix proposal approval status automation#162
ewels merged 1 commit into
mainfrom
fix/proposal-approval-automation

Conversation

@ewels

@ewels ewels commented Aug 14, 2026

Copy link
Copy Markdown
Member

Fix rejected pipeline status updates and prevent duplicate status comments.

Context: @jfy133 mentioned this, see Slack thread.

Details
  • Calculate pipeline status from the issue state and all votes.
  • Serialize workflow runs for each issue.
  • Remove duplicate status comments during the next run.
  • Skip redundant label updates.
  • Add regression tests for the Dragen proposal case.
  • Tests: 62 Jest tests and all pre-commit checks pass.

@github-actions

Copy link
Copy Markdown

📊 Test Coverage Report

Metric Coverage Status
Statements 100%
Branches 89.39% ⚠️
Functions 100%
Lines 100%

Tests: 108 statements covered out of 108

🎉 Excellent coverage!

@ewels
ewels requested review from jfy133 and kenibrewer August 14, 2026 14:05
@ewels
ewels merged commit 52ac44b into main Aug 14, 2026
4 checks passed
@ewels
ewels deleted the fix/proposal-approval-automation branch August 14, 2026 15:21
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