search

rnestler / reboot-arch-btw

Checks if your ArchLinux needs a reboot due to a kernel update
GNU General Public License v3.0
26 stars 4 forks source link

Inconsistent naming #184

Open ShalokShalom opened 2 months ago

ShalokShalom commented 2 months ago

I just searched myself dull, because --session-restart-package is the name of the option on the CLI, and in source, its called session_restart_package - thats probably a me issue, I just wanted to report, that I would appreciate if the names line up.

rnestler commented 1 month ago

Well that's just how the naming conventions are: For command line arguments one uses - to separate words and for variable names one has to use _. I think it would be very unusual to have --sesion_restart_package as a command line argument.

ShalokShalom commented 1 month ago

I think adding a comment or something, would make it searchable.