chore(scripts): remove tmpdir prefix and add dev:ui script#2304
chore(scripts): remove tmpdir prefix and add dev:ui script#2304HugoRCD wants to merge 2 commits into
Conversation
This allow to start the dev server without Nuxi (eve)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
We've flagged this as a potential contribution without a human behind it. We welcome the thoughtful use of AI tools when contributing, but ask all contributors to follow two core principles:
Please review these AI-assisted contribution guidelines and update this contribution if needed. If this was flagged in error, we apologise! 😳 Just let us know. 🙏 |
|
Warning Review limit reached
Next review available in: 11 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. 📝 WalkthroughWalkthroughThe package.json development scripts were updated. The Estimated code review effort: 1 (Trivial) | ~2 minutes Changes
Related issues: None mentioned in the provided information. Related PRs: None mentioned in the provided information. Suggested labels: chore, tooling Suggested reviewers: None specified based on the provided information. Poem 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
package.json (1)
8-8: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winInline env var assignment breaks on native Windows shells.
EVE_BASE_URL=http://127.0.0.1:1 nuxt dev --ui-onlyuses POSIX-style inline env assignment, which fails under nativecmd.exe/PowerShell (only works if run via WSL/git-bash). If Windows dev support matters here, usecross-envfor portability.♻️ Proposed fix using cross-env
- "dev:ui": "EVE_BASE_URL=http://127.0.0.1:1 nuxt dev --ui-only", + "dev:ui": "cross-env EVE_BASE_URL=http://127.0.0.1:1 nuxt dev --ui-only",🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@package.json` at line 8, The dev:ui script currently uses POSIX-style inline environment assignment, which breaks in native Windows shells. Update the package.json script that runs nuxt dev --ui-only to use a portable env setter such as cross-env so EVE_BASE_URL is applied consistently across cmd.exe, PowerShell, and Unix shells.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@package.json`:
- Line 8: The dev:ui script currently uses POSIX-style inline environment
assignment, which breaks in native Windows shells. Update the package.json
script that runs nuxt dev --ui-only to use a portable env setter such as
cross-env so EVE_BASE_URL is applied consistently across cmd.exe, PowerShell,
and Unix shells.
This allow to start the dev server without Nuxi (eve)
🔗 Linked issue
📚 Description