hamza72x / web2app

Turn any web page into a desktop app (but, lightweight ~3MB)
Other
114 stars 6 forks source link

Suggestion: remove positional args #16

Closed pomdtr closed 1 year ago

pomdtr commented 1 year ago

The args vs interactive subcommands are confusing.

What about:

ex:

nativefier_tauri # will show all of the prompts
nativier_tauri --url google.com # will show all the prompts except the url one
hamza72x commented 1 year ago

Thanks for issue. But the main reason for splitting these 2 was to split complete bahavior, not to use within interchange.

pomdtr commented 1 year ago

Since the final action is the same (creating an app), it makes no sense to have two separate commands.

Instead, having a single command that can be called either interactively or non-interactively is a common pattern (npm init is a good example).

hamza72x commented 1 year ago

Thank you, but it's rather subjective choice. And not trying to make it more complex for this simple tool. I hope you don't mind.