chore(ci): Simplify release workflow - #6
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reachedNext included review available in 25 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (3)
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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Code Review ✅ ApprovedSimplifies the release workflow by introducing a OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
There was a problem hiding this comment.
🟢 Approval recommended
The workflow and script changes are straightforward and align with the stated goal; only a minor maintainability refactor was suggested to prevent script drift.
Pull request overview
This PR simplifies the CI and release workflows by introducing a CI-focused release gate script that excludes E2E tests (and therefore removes the need to install Playwright browsers) while keeping the full release validation available via check:release.
Changes:
- Add
check:release:ciscript that runs the full release-quality gate minus E2E. - Update CI and Release GitHub Actions workflows to use
pnpm check:release:ci. - Remove Playwright browser installation steps from CI and Release workflows.
File summaries
| File | Description |
|---|---|
| package.json | Adds check:release:ci to run release checks without E2E. |
| .github/workflows/release.yml | Switches release gate to check:release:ci and drops Playwright browser install. |
| .github/workflows/ci.yml | Switches CI gate to check:release:ci and drops Playwright browser install. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bd678d17a5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Description
What does this PR do?
check:release:ciwith formatting, lint, changeset, peer dependency, security audit, build, type, package validation, bundle size, and unit coverage checks.check:releaseavailable for the full validation sequence, including E2E tests.Why is this change needed?
pnpm test:e2eandpnpm check:release.Type of Change
Testing
check:release:cidiffers fromcheck:releaseonly by excluding the final E2E command.Screenshots (if applicable)
Checklist
Maintainers only (
write/maintain/adminaccess): open the workflow, clickRun workflow, and setpr_numberto this PR number to post/update a coverage comment on this PR