search

vpsfreecz / vpsadminos

Host for Linux system containers based on NixOS, ZFS and LXC
https://vpsadminos.org
MIT License
155 stars 26 forks source link

adding missing osctl command for sub commands in administration container doc page #59

Closed yater closed 1 year ago

yater commented 1 year ago

without running vpsadminos (and while writing the docs) it seem obvious that for the sub commands the command osctl was missing. (btw: why sometimes console is used for syntax highlighting instead of bash?)

aither64 commented 1 year ago

Great to see someone is actually reading it x)

The commands are not wrong though, they work. Having to always write osctl would be annoying, so we have shortcuts for some frequent subcommands, it's described in the man page: https://man.vpsadminos.org/man8/osctl.8.html#command-shortcuts. It's true that it could be mentioned somewhere on the page you edited.

As for console vs bash, I don't remember if there's a reason. I dislike this part, I would like to have shell prompts in front of the commands, so that's it clear those are commands, but I think it used to break the syntax highlighting.

aither64 commented 1 year ago

I've merged your patch as e2b7e5c3a5057bdcf4f71fd7aeb0ebd4863cdf90, it will be better to have osctl there to keep the page consistent. Thanks.