fix(template): use correct heroui-native path in monorepo setups#925
fix(template): use correct heroui-native path in monorepo setups#925bibblebabl wants to merge 1 commit intoAmanVarshney01:mainfrom
Conversation
In turborepo monorepos, bun hoists heroui-native to the root node_modules, so the @source directive in global.css needs to reference ../../node_modules instead of ./node_modules. Convert global.css to a Handlebars template that conditionally sets the correct path based on whether turborepo addon is enabled. Fixes AmanVarshney01#810
|
@bibblebabl is attempting to deploy a commit to the Better T Stack Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
WalkthroughAdded a conditional in the HeroUI Uniwind CSS template to select the appropriate node_modules import path based on whether the turborepo addon is included. When turborepo is present, uses a relative path traversal; otherwise uses the default path. Changes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
Summary
heroui-nativeto the rootnode_modules, so@source './node_modules/heroui-native/lib'inglobal.cssdoesn't resolve — HeroUI classes silently go missingglobal.cssto a Handlebars template (.hbs) that conditionally uses../../node_modules/heroui-native/libwhen the turborepo addon is enabled, and keeps the original./node_modules/heroui-native/libfor non-monorepo setupsFixes #810
Summary by CodeRabbit