Repository navigation
ci: remove redundant settings - #430
lachlancollins wants to merge 2 commits into
Conversation
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Remove explicit setup options .github/workflows/autofix.yml, .github/workflows/pr.yml, .github/workflows/release.yml |
The Vite+ setup steps no longer pass run-install: true or cache: false. |
Estimated code review effort: 1 (Trivial) | ~5 minutes
Change: Other
Possibly related PRs
- TanStack/config#429: This PR sets the Vite+ setup options that this change removes.
Suggested reviewers: schiller-manuel
Merge Risk: 
Minimal · up to 102a4
-
The workflow cleanup preserves installation and caching behavior and is ready to merge after normal checks.
Pre-merge checks: 
4
Finishing Touches
-
Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- 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.
|
View your CI Pipeline Execution ↗ for commit 102a41f
☁️ Nx Cloud last updated this comment at |
commit: |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
🎯 Changes
These options are the defaults, so it's unnecessarily verbose.
See https://github.com/voidzero-dev/setup-vp#inputs
✅ Checklist
pnpm test:pr.🚀 Release Impact
Summary by CodeRabbit