nextui-org / nextui-cli

⌨️ A CLI tool that unlocks seamless NextUI integration.
MIT License
40 stars 14 forks source link

feat: upgrade command add peerDep upgrade #30

Closed winchesHe closed 4 months ago

winchesHe commented 4 months ago

📝 Description

1、Add peerDependencies upgrade

https://github.com/nextui-org/nextui-cli/assets/96854855/49d94176-45af-4085-b6f4-9cd9c399b887

2、Warn when mising

image

3、When component is latest also will check the peerDependencies

image

4、Individual component upgrade

image

✅ Type of change

jrgarciadev commented 4 months ago

@winchesHe could you please check this: CleanShot 2024-04-22 at 16 18 24

We should only display the warning once when it is the same warning

jrgarciadev commented 4 months ago

Same here @winchesHe CleanShot 2024-04-22 at 17 40 05

jrgarciadev commented 4 months ago

I believe we could add a warning message if the peerDependency like tailwindcss is installed but is not the min version

winchesHe commented 4 months ago

@jrgarciadev Above issues i have resolved in this PR: https://github.com/nextui-org/nextui-cli/pull/42, after i merged it, could you try again

winchesHe commented 4 months ago

@jrgarciadev After merged it is work normally

image
winchesHe commented 4 months ago

@jrgarciadev After merged it is work normally

image

Fix the output info align

List

image

Upgrade

image
winchesHe commented 4 months ago

I believe we could add a warning message if the peerDependency like tailwindcss is installed but is not the min version

What about this you think

image
jrgarciadev commented 4 months ago

Hey @winchesHe could you please adjust this as well?

Instead of showing a warning about the missing peerDepedencies we could list them inside the PeerDepedencies alert and install them

CleanShot 2024-04-24 at 10 39 15

jrgarciadev commented 4 months ago

@winchesHe feel free to merge this PR once committed my suggestions

jrgarciadev commented 4 months ago

@winchesHe could you please fix the conflicts?

winchesHe commented 4 months ago

@jrgarciadev Done