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

change single dash long arguments to double dash long arguments #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CMon
Copy link
Contributor

@CMon CMon commented Apr 15, 2024

with threagile version > 0.9.0, a single dash for long arguments do not work anymore

with threagile version > 0.9.0, a single dash for long arguments do not
work anymore
@ezavgorodniy
Copy link
Collaborator

Could you please add more details on exact example when it's not working for you?

Also @joreiche may share more details on using confi file instead of arguments in command line - that's not documented but may improve a usability of the tool

@CMon
Copy link
Contributor Author

CMon commented Apr 16, 2024

sure:

$ docker run --rm -it -v "$(pwd)":/work threagile create-example-model -output /work 
unknown shorthand flag: 'o' in -output

and if i use the double dash:

$ docker run --rm -it -v "$(pwd)":/work threagile create-example-model --output /work

it works even if I also get the help printed as well, same for the -model and --model:

$ docker run --rm -it -v "$(pwd)":/work threagile analyze --output /work -model /work/threagile-example-model.yaml
unknown shorthand flag: 'm' in -model

and with double dash:

$ docker run --rm -it -v "$(pwd)":/work threagile analyze --output /work --model /work/threagile-example-model.yaml
failed to read and analyze model: unable to check risk tracking: Risk tracking references unknown risk (risk id not found) - you might want to use the option -ignore-orphaned-risk-tracking: unencrypted-asset@*

even if the example is not working anymore at least it started with the parsing

@CMon
Copy link
Contributor Author

CMon commented Apr 29, 2024

@ezavgorodniy is there anything else you need as information?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants