search

iafan / goplayspace

Advanced Go Playground frontend written in Go, with syntax highlighting, turtle graphics mode, and more
https://goplay.space
Other
982 stars 61 forks source link

Remind users to update GopherJS first #7

Closed damienstanton closed 7 years ago

damienstanton commented 7 years ago

Since I stumbled on this in #6 and got an inconsistent build/error from gopherjs, it may be helpful to obviate it up front.

iafan commented 7 years ago

I was under impression that, as per 'go get' docs, the go get -u github.com/iafan/goplayspace/... command should install/update the dependencies, so this instruction on updating individual dependencies theoretically is not needed.

With that in mind, I'd suggest using some more verbose wording by e.g. adding the following section right before Credits:

Troubleshooting

If you have trouble compiling the client, please make sure you have the latest version of GopherJS installed by running go get -u github.com/gopherjs/gopherjs (see #6)

If you can update the PR, I'll be happy to merge it.

iafan commented 7 years ago

Thanks, Damien!