Skip to content

fix(ci): run validate-release on pull_request events#8

Merged
omrdev1 merged 2 commits into
mainfrom
fix/ci-trigger-on-pr
Mar 27, 2026
Merged

fix(ci): run validate-release on pull_request events#8
omrdev1 merged 2 commits into
mainfrom
fix/ci-trigger-on-pr

Conversation

@omrdev1
Copy link
Copy Markdown
Owner

@omrdev1 omrdev1 commented Mar 27, 2026

Problem

validate-release was permanently showing "Expected — Waiting for status to be reported" on every PR because the workflow only triggered on push to main and tags — never on pull_request events.

This meant the required status check could never go green, forcing every PR to use bypass rules to merge.

Fix

Add pull_request: branches: [main] trigger to release-check.yml so CI actually runs on PRs and the status check can resolve.

GreenOps E2E added 2 commits March 27, 2026 08:11
The workflow only triggered on push to main and tags, never on PRs.
This caused validate-release to show 'Expected — Waiting for status'
permanently on every PR, blocking the required status check.

Adding pull_request trigger so CI actually runs and can go green.
@github-actions
Copy link
Copy Markdown

🌱 GreenOps Infrastructure Impact

Total Current Footprint: 7.06kg CO2e/month | $126.29/month
Potential Savings: -2.60kg CO2e/month (36.8%) | -$13.87/month
💡 Found 2 optimization recommendations.

Resource Breakdown

Resource Type Region CO2e/month Cost/month Action
aws_instance.web m5.large us-east-1 4.31kg $70.08 💡 View Recommendation
aws_instance.worker m6g.large us-east-1 2.74kg $56.21 💡 View Recommendation

Recommendations

aws_instance.web

  • Current: m5.large in us-east-1
  • Suggested: m6g.large in us-east-1
  • Impact: -1.57kg CO2e/month | -$13.87/month
  • Rationale: Switching from m5.large (x86_64) to m6g.large (ARM64) provides identical vCPU and memory at lower power draw, reducing carbon by 1570g CO2e/month and cost by $13.87/month.

aws_instance.worker

  • Current: m6g.large in us-east-1
  • Suggested: m6g.large in us-west-2
  • Impact: -1.03kg CO2e/month | +$0.00/month
  • Rationale: Moving m6g.large from us-east-1 to US West (Oregon) (us-west-2) reduces grid carbon intensity from 384.5g to 240.1g CO2e/kWh, saving 1030g CO2e/month saving $0.00/month.

Emissions calculated using the Open GreenOps Methodology Ledger (v1.1.0). Scope 2 operational emissions only — embodied carbon and water are not tracked. Math is MIT-licensed and auditable. Analysed at 2026-03-27T08:13:09.830Z. Learn more.

🏢 Managing green-ops across dozens of repositories? Upgrade to GreenOps Dashboard to aggregate CI/CD carbon data natively.

@omrdev1 omrdev1 merged commit d63d64f into main Mar 27, 2026
2 checks passed
@omrdev1 omrdev1 deleted the fix/ci-trigger-on-pr branch March 27, 2026 08:18
omrdev1 added a commit that referenced this pull request May 19, 2026
* fix(ci): run validate-release on pull_request events

The workflow only triggered on push to main and tags, never on PRs.
This caused validate-release to show 'Expected — Waiting for status'
permanently on every PR, blocking the required status check.

Adding pull_request trigger so CI actually runs and can go green.

* chore: rebuild dist with v0.2.2 version string

---------

Co-authored-by: GreenOps E2E <greenops-e2e@test.local>
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