Skip to content

Refactor conditions for preview release triggers - #2550

Open
UnschooledGamer wants to merge 5 commits into
mainfrom
UnschooledGamer-patch-1
Open

Refactor conditions for preview release triggers#2550
UnschooledGamer wants to merge 5 commits into
mainfrom
UnschooledGamer-patch-1

Conversation

@UnschooledGamer

Copy link
Copy Markdown
Member

Clarifies PR workflow execution and approval process,
adds allow-unsafe-pr-checkout to nightly-build checkouts when running from PR context.

Formats conditions for preview release triggers.

@UnschooledGamer UnschooledGamer self-assigned this Jul 28, 2026
@UnschooledGamer
UnschooledGamer added this pull request to the merge queue Jul 28, 2026
@UnschooledGamer
UnschooledGamer removed this pull request from the merge queue due to a manual request Jul 28, 2026
Comment thread .github/workflows/on-demand-preview-releases-PR.yml Outdated
@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR refines preview-release trigger conditions and explicitly permits PR-head checkout in the reusable nightly build.

  • Handles bypass-label checks through the pull request’s label collection during synchronize events.
  • Restricts the standard preview label check to labeled events.
  • Documents the approval requirement before executing PR code.
  • Enables allow-unsafe-pr-checkout when the nightly workflow runs in PR context.

Confidence Score: 5/5

The PR appears safe to merge.

The previously reported synchronize-event label lookup is corrected by checking the pull request’s labels, and no blocking failure remains.

Important Files Changed

Filename Overview
.github/workflows/on-demand-preview-releases-PR.yml Refactors event-specific label conditions and correctly checks the bypass label from the pull request payload on synchronize events.
.github/workflows/nightly-build.yml Documents PR-code approval expectations and enables explicitly approved PR-head checkout for PR builds.

Reviews (3): Last reviewed commit: "Update .github/workflows/on-demand-previ..." | Re-trigger Greptile

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@UnschooledGamer

This comment was marked as resolved.

1 similar comment
@UnschooledGamer

This comment was marked as resolved.

@UnschooledGamer
UnschooledGamer requested a review from a team July 28, 2026 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant