Skip to content

ci: adds pull-request-age-gate - #145

Open
bmuenzenmeyer wants to merge 2 commits into
mainfrom
age-gate
Open

bmuenzenmeyer wants to merge 2 commits into
mainfrom
age-gate

Conversation

@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

just like nodejs.org

@bmuenzenmeyer
bmuenzenmeyer requested a review from a team as a code owner September 23, 2026 02:32
Copilot AI lite review requested due to automatic review settings September 23, 2026 02:32
@vercel

vercel Bot commented Sep 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
nodejs-learn Ready Ready Preview Sep 23, 2026 11:02am UTC

Request Review

@github-actions

Copy link
Copy Markdown

👋 Codeowner Review Request

The following codeowners have been identified for the changed files:

Team reviewers: @nodejs/web-infra

Please review the changes when you have a chance. Thank you! 🙏

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Three moderate issues and one nit remain unresolved.

Review effort: Lite
Findings: None

What changed in this PR

Adds a 48-hour pull-request age gate with hourly reevaluation and a fast-track bypass label.

Changes:

  • Adds pull-request and scheduled triggers.
  • Uses pinned, permission-scoped actions.
  • Configures age-gate and bypass behavior.
File Summary Findings
.github/​workflows/​pull-request-age-gate.yml Defines the age-gate workflow and security settings. Moderate (1 vote): Concurrent runs can overwrite gate results; fork workflows cannot write checks; the fast-track label does not exist. Nit (1 vote): The settings link targets the wrong repository.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/pull-request-age-gate.yml Outdated
Co-authored-by: Michael Esteban <mickel13@gmail.com>
Signed-off-by: Matt Cowley <me@mattcowley.co.uk>

@MattIPv4 MattIPv4 left a comment

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.

This branch was successfully deployed

1 active deployment
Preview 74f349f0 Deployed Sep 23, 2026 by vercel[bot]
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.

6 participants