-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
x/tour: welcome/3 slide missing instructions for installing Go #34585
Labels
Documentation
Issues describing a change to documentation.
FrozenDueToAge
NeedsFix
The path to resolution is known, but the work has not been done.
Milestone
Comments
Change https://golang.org/cl/197857 mentions this issue: |
gopherbot
pushed a commit
to golang/website
that referenced
this issue
May 26, 2021
Re-add the link to installing Go on the welcome/3 slide, because it's a pre-requisite for being able to run go get. This was included before, but got lost in a refactor after the tour binary moved out of the Go distribution. Additionally, make it more clear that installing the tour locally is an optional step that only needs to be done if one wishes to run the tour offline; it's not mandatory to get through the tour. Running the tour online has a lower overhead, and should help keep the tour accessible to more people. Fixes golang/go#34585 Updates golang/go#24819 Change-Id: If5f51e9e876722a86afca054bf78428f78bb3076 Reviewed-on: https://go-review.googlesource.com/c/tour/+/197857 Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com> X-Tour-Commit: 45121acd746573507cdaaf902958dba082f7c25b
MK825
added a commit
to MK825/website
that referenced
this issue
Oct 18, 2022
Re-add the link to installing Go on the welcome/3 slide, because it's a pre-requisite for being able to run go get. This was included before, but got lost in a refactor after the tour binary moved out of the Go distribution. Additionally, make it more clear that installing the tour locally is an optional step that only needs to be done if one wishes to run the tour offline; it's not mandatory to get through the tour. Running the tour online has a lower overhead, and should help keep the tour accessible to more people. Fixes golang/go#34585 Updates golang/go#24819 Change-Id: If5f51e9e876722a86afca054bf78428f78bb3076 Reviewed-on: https://go-review.googlesource.com/c/tour/+/197857 Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com> X-Tour-Commit: 45121acd746573507cdaaf902958dba082f7c25b
KimuraDanaka119
added a commit
to KimuraDanaka119/go-tour-jp
that referenced
this issue
Jun 4, 2023
Re-add the link to installing Go on the welcome/3 slide, because it's a pre-requisite for being able to run go get. This was included before, but got lost in a refactor after the tour binary moved out of the Go distribution. Additionally, make it more clear that installing the tour locally is an optional step that only needs to be done if one wishes to run the tour offline; it's not mandatory to get through the tour. Running the tour online has a lower overhead, and should help keep the tour accessible to more people. Fixes golang/go#34585 Updates golang/go#24819 Change-Id: If5f51e9e876722a86afca054bf78428f78bb3076 Reviewed-on: https://go-review.googlesource.com/c/tour/+/197857 Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
Documentation
Issues describing a change to documentation.
FrozenDueToAge
NeedsFix
The path to resolution is known, but the work has not been done.
The first slide of the tour welcomes people and explains how to change slides. The second slide mentions the choice of spoken languages. The third slide strongly suggests running the tour offline.
By the third slide, the tour assumes users already have Go installed (and/or know how to install Go). This isn't valid for users who are being introduced to Go by being linked to the tour. It can cause unnecessary confusion as demonstrated by the experience reported in this comment.
That slide used to have instructions for installing Go, but they were lost as part of deployment changes in CL 141857. /cc @andybons
To fix this, we should include a link to https://golang.org/doc/install and/or https://golang.org/dl/ before the first mention of
go get
in the tour. It should also be helpful and a good teaching opportunity to point out where thetour
binary ends up aftergo get
runs successfully.This issue is factored out from #24819, which was about a formatting bug that is resolved by now.
The text was updated successfully, but these errors were encountered: