zuixinwang comments

Results 9 comments of


                                            zuixinwang

chore(cli): optimize updateNotifier

Aha, I move the update code to the top of the main function. 😃 Can this PR be merged? 👀 @segunadebayo

[VaTreeView] Array doesn't update probably

When checking one node, all descendants' selected states should be recalculated, and all ancestors' indeterminate states and selected states should be recalculated too. https://github.com/epicmaxco/vuestic-ui/blob/32e8f62b49c277f15561adcab2b8ae02eefc6590/packages/ui/src/components/va-tree-view/hooks/useTreeView.ts#L77

A way to override `--version`

Can you provide a reproduction please 😄

main command's positional arg's value is the same as a subCommand name

Maybe it should not have positional args if subCommands exist Should it have a validation?

unbuild preset and build instructions

Good idea. Maybe unbuild should have better development documentation first. 🐕

Improve arg help usage message (placeholder)

The positional arg should be placed before other args and should not exceed one If there is more than one, only one positional arg should be retained, and the rest...

Setup a default behavior when no subcommand used

I think a default subcommand is unnecessary because, without it, the main command would run 👀 Perhaps it's a good idea not to run the run function of the main...

Create alias for subcommand

it's a good idea, but the help output should also be considered

Create shared args for all `subCommands` in main

![image](https://github.com/user-attachments/assets/2424654d-eb9b-4bcc-8c48-571b798fb1b5) maybe you can call the `generateArgs` function to do this outside `citty`