Skip to content

Trial: review apps from control-plane-flow PR 511 without action copies (do not merge) - #860

Closed
justin808 wants to merge 1 commit into
masterfrom
justin808-claude/510-trial-upstream-actions
Closed

justin808 wants to merge 1 commit into
masterfrom
justin808-claude/510-trial-upstream-actions

Conversation

@justin808

Copy link
Copy Markdown
Member

🤖 Claude Code · Anthropic · claude-opus-5-5 · high

Temporary trial. Do not merge.

This branch tests shakacode/control-plane-flow#511 on a real app. It pins the review-app deploy and delete wrappers to that PR's commit and removes .github/actions/cpflow-*, to show a review app deploys and deletes with no action copies.

bin/check-cpflow-review-app-contract is expected to fail here: it requires the 6.0.0 pin and the copies. The branch and this PR will be closed after the trial.

…copies

Temporary test branch for shakacode/control-plane-flow#511. Pins the
review-app deploy and delete wrappers to that PR's commit and removes
.github/actions/cpflow-*. Not for merge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🚀 Quick Review App Commands

Welcome! Here are the commands you can use in this PR:
They require the repository to have cpflow review apps configured, including the CPLN_TOKEN_STAGING secret.

+review-app-deploy

Deploy your PR branch for testing.

+review-app-delete

Remove the review app when done.

+review-app-help

Show detailed instructions, environment setup, and configuration options.

Comment +review-app-help for full setup details.

@github-actions

Copy link
Copy Markdown

No description provided.

@coderabbitai

coderabbitai Bot commented Oct 10, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

contains(fromJson('["+review-app-deploy","+review-app-deploy\n","+review-app-deploy\r\n"]'), github.event.comment.body) &&
contains(fromJson('["OWNER","MEMBER","COLLABORATOR"]'), github.event.comment.author_association))
uses: shakacode/control-plane-flow/.github/workflows/cpflow-deploy-review-app.yml@68b5d83152c02a70eb98ffdc7fd2072e28265fef # v6.0.0
uses: shakacode/control-plane-flow/.github/workflows/cpflow-deploy-review-app.yml@da38e257c6256899e3ab35b0a3bc5d9e02b9934c # control-plane-flow PR 511 trial

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Trial PR (marked do-not-merge), so only notes:

  • This passes CPLN_TOKEN_STAGING and REVIEW_APP_DOCKER_BUILD_SECRETS to a commit from an unmerged upstream PR. The SHA pin is good, but review that commit (da38e25) before running it. It's a force-pushable PR ref until merged, so confirm the SHA is reachable from a protected branch or tag before relying on it.
  • bin/check-cpflow-review-app-contract will fail by design, as the description says. Make sure CI doesn't block unrelated work while this is open.
  • The trial comment replaces the # v6.0.0 pin comment. Any tooling that reads that comment, such as Dependabot or the contract script, will no longer recognize the pin.
  • Merging this would remove the local actions and leave master depending on an unmerged commit. The description already says not to merge, so closing after the trial is the right outcome.

@github-actions

Copy link
Copy Markdown

No description provided.

@github-actions

github-actions Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

🎉 Deploy Complete!

Open Review App

Deployment successful for PR #860, commit 471504b

🎮 Control Plane Console
📋 View Completed Action Build and Deploy Logs

@github-actions
github-actions Bot temporarily deployed to review/qa-react-webpack-rails-tutorial-860 October 11, 2026 02:21 Destroyed
@github-actions

Copy link
Copy Markdown

No description provided.

@github-actions

github-actions Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

✅ Review App Deleted

Review app for PR #860 is deleted

🎮 Control Plane Console
📋 View Workflow Logs

@justin808 justin808 closed this Oct 11, 2026
@github-actions

Copy link
Copy Markdown

No description provided.

@github-actions

github-actions Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

✅ Review App Deleted

Review app for PR #860 is deleted

🎮 Control Plane Console
📋 View Workflow Logs

@justin808
justin808 deleted the justin808-claude/510-trial-upstream-actions branch October 11, 2026 03:52

This branch was successfully deployed

No deployments
review/qa-react-webpack-rails-tutorial-860 — 471504b7 Deployed Oct 11, 2026 by github-actions[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.

1 participant