-
Notifications
You must be signed in to change notification settings - Fork 128
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
Make Warning more prominent #5
Comments
Hm; but what would you like me to say? And where? I cannot for now think of a way that this could be still useful as it is, while at the same time not being potentially dangerous. That on the technical side. If you feel just changing the message would be enough: where and how would you suggest to put it? If I put it before I even explain what it is, I feel I could scare virtually everybody away from even trying it, by projecting an ominous first impression. I tried to put it quite early, just after explaining how to enter the command into the app... Or should I put it straight by the title of the app? How to make it prominent enough? "up - a pipe building app, DANGEROUS, DON'T USE IT"? One thing I'm starting to think of, is that maybe it's not a good idea to use up with commands that could change any permanent state of something (such as writing to files, modifying/deleting them, accessing external web services, etc...). That might be the main message I'd like to try to publish... but not 100% sure about it yet; I'm just starting to think about it; still in an early stage of trying to collect initial thoughts after the public release, and still getting new feedback. What I'm trying to also say, is that I actually don't think it's that much more dangerous than xargs or bash. In particular, I haven't yet imagined a command one could easily type accidentally, that could be really disastrous. Even after typing "rm", you still need to type something that you want to remove. I can't yet see how this could be done really accidentally. And again, you usually don't pipe anything into "rm". One may want to try doing "xargs rm"; but then again, I don't see how it'd be easy to do accidentally as of now? And if I were to do this purposefully, I'd be careful even when writing this in regular bash; I wouldn't want to try doing it in up at any cost... like I wouldn't try driving a ferrari through a minefield... With all that said, I'm totally into trying to invent ways to make it safer. I'm super open to ideas, already collecting some in a notebook, will try to think what to do with them further down the road. (E.g. a shortcut key for pausing/unpausing the "immediate execution" mode.) None of them seem 100% bullet-proof however, unfortunately. But neither bash/xargs are... Hm; actually the pausing/unpausing seems a reasonable cop-out, maybe... then one can choose the preferred mode... maybe even "require enter to execute" would be good enough for typical usage? Even if somewhat slower... |
See also: #8 |
|
I think it would be fine to merge with #8 |
Ok, cool, thanks! |
At the risk of ignoring tongue in cheekiness; this warning seems too buried. Also there is a world of difference between "being careful using rm or dd" vs having a simple typo immediately be run as a command vs having to type something fully before hitting enter. I understand the impulse of closing this loop of iterate and see results; doing it this way though seems beyond dangerous.
The text was updated successfully, but these errors were encountered: