ltopers icon indicating copy to clipboard operation
ltopers copied to clipboard
verified publisher icon amiaopensource

README - update writelto, attempts to fix #169

Open kieranjol opened this issue 6 years ago • 9 comments

I'm almost certain that this is fairly wrong in a few ways, but maybe it'll get the ball rolling. Any edits/fixes are very welcome..

kieranjol avatar Jan 22 '19 20:01 kieranjol

Lol I'm sorry, I'm going pretty blind here.looks like your PR would make this redundant as the docs would be correct of that was merged?

kieranjol avatar Jan 22 '19 21:01 kieranjol

I think that even after that PR is merged, the docs still should be updated along the lines you are doing! I see that PR more as a check against data not being entered initially. People should probably know that they need to add both those values for a complete executable command.

privatezero avatar Jan 22 '19 21:01 privatezero

I agree, maybe the all caps, with an example command to make it extra clear?

kieranjol avatar Jan 22 '19 21:01 kieranjol

I think that makes sense!

privatezero avatar Jan 22 '19 22:01 privatezero

ping

retokromer avatar Jul 12 '19 16:07 retokromer

ping ping

retokromer avatar Jan 26 '20 09:01 retokromer

Has this PR not been superseded by https://github.com/amiaopensource/ltopers/commit/ed8ccabe9e5e09bf578e3cf2f043dc48b84fd4fb?

retokromer avatar Feb 23 '20 08:02 retokromer