Skip to content

Guard CI minutes with a timeout, draft skip, and concurrency cancel - #21

Closed
mgballou wants to merge 1 commit into
mainfrom
ci/guard-actions-minutes
Closed

mgballou wants to merge 1 commit into
mainfrom
ci/guard-actions-minutes

Conversation

@mgballou

@mgballou mgballou commented Oct 3, 2026

Copy link
Copy Markdown
Owner
  • Added timeout-minutes: 15 to the check job in ci.yml
  • Added if: github.event.pull_request.draft == false to check so draft PRs skip CI
  • Added ready_for_review to pull_request.types so marking a draft ready triggers the run
  • Added workflow-level concurrency keyed on workflow and ref with cancel-in-progress: true

@netlify

netlify Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for dreadmajesty ready!

Name Link
🔨 Latest commit 3c3abad
🔍 Latest deploy log https://app.netlify.com/projects/dreadmajesty/deploys/6ac0f7b97592e20008302793
😎 Deploy Preview https://deploy-preview-21--dreadmajesty.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@mgballou

mgballou commented Oct 3, 2026

Copy link
Copy Markdown
Owner Author

Closing in favor of #19. It makes the same changes (draft skip, timeout, concurrency, ready_for_review) and also drops the push trigger, adds the gate job, and now sets read-only permissions.

@mgballou mgballou closed this Oct 3, 2026
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