Skip to content

ci: publish per-PR preview builds to pkg.pr.new - #1535

Open
l2ysho wants to merge 2 commits into
masterfrom
claude/pkg-pr-new-cli-82c8ab
Open

l2ysho wants to merge 2 commits into
masterfrom
claude/pkg-pr-new-cli-82c8ab

Conversation

@l2ysho

@l2ysho l2ysho commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Opt-in via the `cli-preview` label. Phase 1 of #1357.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@l2ysho l2ysho added the t-builders Issues owned by the Builders team. label Oct 30, 2026
Drops the `cli-preview` label gate. Forks and bot authors stay excluded.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@pkg-pr-new

pkg-pr-new Bot commented Oct 30, 2026

Copy link
Copy Markdown

Open in StackBlitz

npx https://pkg.pr.new/apify-cli@1535

commit: 36c5551

@l2ysho
l2ysho requested a review from apify-factory October 30, 2026 10:31
@l2ysho l2ysho added the adhoc Ad-hoc unplanned task added during the sprint. label Oct 30, 2026

@apify-factory apify-factory 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.

Looks good, nothing blocking.

🏭 Factory review — ℹ️ not auto-approved

This change was reviewed but cannot be auto-approved: a gate failed, so the review cannot end in an approval.

check result
Static gates ❌ 13/16 — pr-open-and-ready, file-extensions, deny-globs
Review findings ✅ none
Reviewer escalation ❌ Adds a new CI workflow that publishes PR builds to an external service (pkg.pr.new) and a new devDependency, which are supply-chain and infrastructure changes.
Juror 1 — claude-sonnet-5-5, adversarial ⏭️ skipped — a gate failed
Why it was not auto-approved

Failed gates:

  • pr-open-and-ready: PR is a draft
  • file-extensions: disallowed extension: .github/workflows/pr_preview.yaml, package.json, pnpm-lock.yaml
  • deny-globs: .github/workflows/pr_preview.yaml matches .github/**; package.json matches **/package.json; pnpm-lock.yaml matches **/pnpm-lock.yaml

The comments above are the review itself; this section only explains the approval decision. Push a change and request @apify-factory again to re-run it, or ask a human to review.

Reviewed 36c5551de · workflow run

@l2ysho
l2ysho marked this pull request as ready for review October 30, 2026 12:28
@l2ysho
l2ysho requested a review from DaveHanns as a code owner October 30, 2026 12:28

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-builders Issues owned by the Builders team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants