Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

better CLI #172

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

better CLI #172

wants to merge 2 commits into from

Conversation

treaki
Copy link

@treaki treaki commented Aug 4, 2024

hi,

i dont know much about python, probably i am to old for that, i am more the java or hardcore c guy, but what i know is how a good cli should be, and that is not what we got here.

so here is one, still quick and dirty, that dose what i miss on the build in one, a way to use it in conjobs and other scripts which is hardly possible if partly parameters are asked as parameters and part;y interactively. There is no reason why that shouldnt be possible.

it is arguable if this is the nicest way of if it should better be something like -i {ip adress} so you can order them however you want and it is more posix/gnu like but for me that works fine that way.

i hope i can make some projects of other users out there a bit easier when sharing this, i right now build it into another independent file in the main dir to dont brake anything but i have no problem if someone would merge it into the main cli...

i fear this will be a one shot because it solves my problem and i will not find much time to maintain it but maybe not and i will be able to find some spare time, we will see...

regards

T

treaki and others added 2 commits August 4, 2024 21:38
better cli without annoying interactive requests for every value
@sbidy sbidy self-assigned this Dec 3, 2024
@sbidy sbidy added the enhancement New feature or request label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants