Skip to content

fix(ci): fetch all open PRs in Netlify cleanup workflow#17246

Merged
wackerow merged 1 commit into
devfrom
fix-cleanup-deploys
Feb 5, 2026
Merged

fix(ci): fetch all open PRs in Netlify cleanup workflow#17246
wackerow merged 1 commit into
devfrom
fix-cleanup-deploys

Conversation

@pettinarip
Copy link
Copy Markdown
Member

Summary

  • Fix pagination bug in Netlify preview cleanup workflow
  • gh pr list defaults to 30 results, but we have 61+ open PRs
  • This caused preview deploys for PRs beyond the first 30 to be incorrectly deleted

Test plan

  • Verify workflow runs successfully on schedule
  • Confirm no preview deploys are incorrectly deleted for open PRs

gh pr list defaults to 30 results, causing PRs beyond the first 30 to
be incorrectly identified as closed. With 61+ open PRs, this led to
preview deploys being deleted for PRs that were still open.
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 5, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit f165da0
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/6984e8f853c7860008e5beb7
😎 Deploy Preview https://deploy-preview-17246.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 56 (🔴 down 3 from production)
Accessibility: 94 (no change from production)
Best Practices: 100 (🟢 up 1 from production)
SEO: 100 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the tooling 🔧 Changes related to tooling of the project label Feb 5, 2026
echo "Found $PREVIEW_COUNT preview/branch deploys"

OPEN_PRS=$(gh pr list --state open --json number --jq '.[].number')
OPEN_PRS=$(gh pr list --state open --limit 1000 --json number --jq '.[].number')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh boy, hope it never gets remotely close to this limit 😆

@wackerow wackerow merged commit 707a681 into dev Feb 5, 2026
12 checks passed
@wackerow wackerow deleted the fix-cleanup-deploys branch February 5, 2026 20:44
@wackerow wackerow mentioned this pull request Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling 🔧 Changes related to tooling of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants