-
Notifications
You must be signed in to change notification settings - Fork 150
Add --give-projects-full-access-to-my-computer-including-ability-to-install-malware #1112
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
base: master
Are you sure you want to change the base?
Conversation
|
can you choose another name other than |
|
i think its a good name but what do you propose |
|
i think the intent of the long name is to let users ABSOLUTELY KNOW that they're enabling a dangerous option that they really shouldn't use |
this name is much better and shorter anyways |
|
I think you should keep it as the long name I agree with @CST1229 |
most command line flags are meant to be short and easy to understand. but this the most ridiculous name for a command line flag. if all of them where as long as this, it would be super frustrating, and might even break the PATH limit. on the other hand, it alerts users to ABSOLUTLEY NOT use this flag for security. |
9cccadd to
8d5ad91
Compare
|
i'm not sure if that's a good idea |
8d5ad91 to
8603f1c
Compare
#1110
Must be enabled by providing a command line flag and user must click agree to a prompt that looks like: