chore(ci): update workflows for pull request and main branch snapshots#71
chore(ci): update workflows for pull request and main branch snapshots#71Adammatthiesen wants to merge 3 commits intomainfrom
Conversation
|
📝 WalkthroughWalkthroughThe PR refactors the CI/CD pipeline to adopt an external preview workflow system from withstudiocms/automations, replacing the previous snapshot release approach. Changes include updating the changeset base branch reference, restructuring PR workflows with conditional bot-based routing, adding a preview job to the main push workflow, and adjusting dependency ignore configurations accordingly. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
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. Comment |
commit: |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/workflows/ci-pr-snapshots.yml:
- Around line 5-9: Edit the paths-ignore list in the ci-pr-snapshots.yml
workflow: remove the ".github/**/**" entry so workflow-only PRs are not skipped,
and replace the incorrect glob "**/**.md" with the correct "**/*.md"; keep the
other ignore entries intact and ensure YAML indentation/format remains valid.
In @.github/workflows/ci-push-main.yml:
- Around line 75-85: The preview-main job currently lacks a fork-guard and will
run on fork pushes; mirror the mergebot guard by adding the same "if" condition
used there to the preview-main job (identify the job by name preview-main) so it
only runs for the upstream owner (e.g., github.repository_owner ==
'withstudiocms' or the same expression used in mergebot); update the
preview-main job's top-level config to include that if-condition and ensure
secrets/uses remain unchanged.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
.changeset/config.json.github/workflows/ci-pr-snapshots.yml.github/workflows/ci-push-main.ymlknip.config.tspackage.json
💤 Files with no reviewable changes (1)
- package.json
Replace current workflows for snapshots with matching ones from main repo
Summary by CodeRabbit