search

tremorlabs / tremor-cli

10 stars 5 forks source link

Missing dependency on running init #10

Closed rajdip-b closed 9 months ago

rajdip-b commented 1 year ago

The dependency @headlessui/tailwindcss is not added by the npx @tremor/cli@latest init command. Am I doing something wrong or this is a bug?

severinlandolt commented 1 year ago

No, you are right here. This should be added! I am working on an improved theming structure, planning it to release it this month. Would you be open to contribute once the new structure is in place?

rajdip-b commented 1 year ago

No, you are right here. This should be added! I am working on an improved theming structure, planning it to release it this month. Would you be open to contribute once the new structure is in place?

Would be happy to help!!

severinlandolt commented 10 months ago

Hey there @rajdip-b

Sorry for the long wait! I merged some PR's and coded up some new stuff, including the missing dependencies.

Would be great to have you giving it a try, just run:

npx @tremor/cli@beta init
rajdip-b commented 10 months ago

Great! I will surely check this out.

severinlandolt commented 10 months ago

@rajdip-b Did it work as expected?

rajdip-b commented 9 months ago

Hey @severinlandolt, sorry for the late response! I just tried it now, and it seems to work fine. Although, I would like to propose a change, if it doesn't break the flow.

image

Currently, the CLI can't detect what kind of project it is init-ing in. So, it would be nice to have auto-detection feature (from package.json / next config / vite config). If the kind cant be determined, we fall back to the menu driven approach.

P.S. Guess we can close this issue!