feat(setup github): add --with-plan option for PR preview#1047
Merged
Conversation
🦋 Changeset detectedLatest commit: a83094d The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
⚡ pkg.pr.new@tailor-platform/sdk@tailor-platform/create-sdk
|
This comment has been minimized.
This comment has been minimized.
Embed the SHA and version comment directly in the generated workflow YAML so Renovate's customManager picks it up, and remove the TS-side ACTIONS_REF / ACTIONS_VERSION constants. Also fix the customManager that wasn't actually matching: wrap the managerFilePatterns in /.../ (Renovate v40+ treats unwrapped values as globs), add a subpath capture for composite action paths (tailor-platform/actions/deploy etc.), and set an autoReplaceStringTemplate that preserves the uses: prefix and subpath. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Merge plan-job.yml into deploy.workflow.yml using marker comments and strip the section when --with-plan is omitted, instead of injecting a separate snippet. Simpler control flow and a single source of truth for the workflow shape. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Code Metrics Report (packages/sdk)
Details | | main (f3674c0) | #1047 (b471242) | +/- |
|--------------------|----------------|-----------------|-------|
- | Coverage | 60.9% | 60.9% | -0.1% |
| Files | 358 | 358 | 0 |
| Lines | 12217 | 12225 | +8 |
+ | Covered | 7448 | 7452 | +4 |
+ | Code to Test Ratio | 1:0.4 | 1:0.4 | +0.0 |
| Code | 80084 | 80133 | +49 |
+ | Test | 32929 | 32958 | +29 |Code coverage of files in pull request scope (69.5% → 66.6%)
SDK Configure Bundle Size
Runtime Performance
Type Performance (instantiations)
Reported by octocov |
📖 Docs Consistency Check✅ No inconsistencies found between documentation and implementation. Checked areas: CLI Documentation
Implementation Verification
Test Coverage
Workflow Behavior
Cross-References
|
remiposo
approved these changes
May 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add
--with-plan/-poption tosetup githubcommand to include a plan job in the generated workflow.Changes
plan-job.ymlsnippet for the plan jobdeploy.workflow.ymlto support combined workflow (deploy + plan)--with-planoption to the commandpull_request: branches: [main]for plan,push: branches: [main]for deployUsage
tailor-sdk setup github \ --workspace-name my-app \ --workspace-region asia-northeast \ --organization-id org-123 \ --folder-id folder-456 \ --with-plan # Include plan job for PR previewsGenerated Workflow
Single workflow file
tailor-<workspace-name>.ymlwith:Dependencies
Requires
tailor-platform/actions/planaction: