Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(vite): ensures the paths are correctly formatted for all os #534

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

shivarm
Copy link
Contributor

@shivarm shivarm commented Jan 16, 2025

paths should be correctly formatted, and the import should resolve without errors on all operating systems. This ensures compatibility and avoids issues related to incorrect path formatting

Fixes: #511

Copy link

vercel bot commented Jan 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zudoku-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 0:36am
zudoku-ship-happens ✅ Ready (Inspect) Visit Preview Jan 21, 2025 0:36am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
zudoku-www ⬜️ Skipped (Inspect) Jan 21, 2025 0:36am

@vercel vercel bot temporarily deployed to Preview – zudoku-www January 16, 2025 02:21 Inactive
Copy link

vercel bot commented Jan 16, 2025

@shivam-sharma7 is attempting to deploy a commit to the Zuplo WWW Team on Vercel.

A member of the Team first needs to authorize it.

@shivarm shivarm requested a review from mosch January 16, 2025 09:24
@vercel vercel bot temporarily deployed to Preview – zudoku-www January 16, 2025 23:33 Inactive
@vercel vercel bot temporarily deployed to Preview – zudoku-www January 20, 2025 14:55 Inactive
@vercel vercel bot temporarily deployed to Preview – zudoku-www January 21, 2025 06:01 Inactive
@vercel vercel bot temporarily deployed to Preview – zudoku-www January 21, 2025 08:24 Inactive
@shivarm shivarm marked this pull request as draft January 21, 2025 08:39
@vercel vercel bot temporarily deployed to Preview – zudoku-www January 21, 2025 08:54 Inactive
@shivarm shivarm marked this pull request as ready for review January 21, 2025 08:54
@shivarm
Copy link
Contributor Author

shivarm commented Jan 21, 2025

@mosch actually @dan-lee made some change recent which also not worked for me, you can verify here: https://github.com/shivarm/zudoku/blob/1a3204d76a676965e6a5daa80e4dd59d10b35d59/packages/zudoku/src/vite/plugin-api.ts#L178

@dan-lee
Copy link
Contributor

dan-lee commented Jan 21, 2025

Ah sorry @shivarm, I wanted to fix this in the other PR but it slipped my mind 😵‍💫

@shivarm
Copy link
Contributor Author

shivarm commented Jan 21, 2025

Ah sorry @shivarm, I wanted to fix this in the other PR but it slipped my mind 😵‍💫

No problem let's merge this.

Copy link

nx-cloud bot commented Jan 21, 2025

View your CI Pipeline Execution ↗ for commit a2483e8.

Command Status Duration Result
nx run ship-happens:build ✅ Succeeded 48s View ↗
nx run docs:build ✅ Succeeded 38s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-21 12:35:06 UTC

@dan-lee
Copy link
Contributor

dan-lee commented Jan 21, 2025

Thanks 🙏

@dan-lee dan-lee merged commit 1ecbd0b into zuplo:main Jan 21, 2025
7 checks passed
@shivarm shivarm deleted the issue_511 branch January 21, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

Vite is unable to resolve the specified import path
3 participants