unitctl: rename app to apps

nginx / unit

NGINX Unit - universal web app server - a lightweight and versatile open source server that simplifies the application stack by natively executing application code across eight different programming language runtimes.

https://unit.nginx.org

Apache License 2.0

5.37k stars 323 forks source link

unitctl: rename app to apps #1424

Closed javorszky closed 1 week ago

javorszky commented 1 week ago

This PR achieves three separate things:

cleans up some grouping of args with its commands to separate it from the next commands
renamed the app to apps. Clap picks up the name of the enum as the subcommand name absent a command argument. I chose to rename the enum rather than add an option
adds the apps line to the readme

It does not fix the whole of the readme, that's a separate issue and PR.

Closes #1423

javorszky commented 1 week ago

I think it's probably worth cleaning up the references to singular unitctl app in the README.md as part of this PR

Fixed in f5a4e44

javorszky commented 1 week ago

Please squash together the 3 commits that aren't whitespace/reordering lines, otherwise the net effect LGTM.

squashed

javorszky commented 1 week ago

@ac000 I fixed the commit subjects 🙂

ac000 commented 1 week ago

Rebase with master.