gort icon indicating copy to clipboard operation
gort copied to clipboard

Documentation: gort digispark install does more than just downloading the software

Open christophberger opened this issue 7 years ago • 7 comments

Hi,

Minor issue.

gort's help text says:

gort digispark install # installs software to upload firmware to Digispark
gort digispark upload [littlewire] # uploads firmware to Digispark

but actually go digispark install does both steps in one: install software to upload firmware to Digispark, AND upload firmware (littlewire) to Digispark.

So the actual behavior is not in line with the help text.

christophberger avatar Jul 20 '17 18:07 christophberger

Hi @christophberger you are indeed correct, seems like this line of code https://github.com/hybridgroup/gort/blob/master/commands/digispark.go#L42 should be removed.

Want to create a PR with that change? :)

deadprogram avatar Jul 25 '17 21:07 deadprogram

Sure. Shall I create the PR against the dev branch, since the fix does change the behavior?

christophberger avatar Jul 26 '17 08:07 christophberger

Yes, please, and thank you!

deadprogram avatar Jul 26 '17 08:07 deadprogram

Looks like dev is behind master. digispark.go still contains the old invalid urls to littlewire.cc on dev. What's the best way to move forward?

christophberger avatar Jul 26 '17 09:07 christophberger

I just rebased master into dev, so you should be good to go now. Sorry about that.

deadprogram avatar Jul 26 '17 09:07 deadprogram

No problem. I created a PR just now.

christophberger avatar Jul 26 '17 10:07 christophberger

Thanks for the PR. I will leave the issue open until the next release of Gort.

deadprogram avatar Jul 26 '17 10:07 deadprogram