Skip to content
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

tour: link to time doc for After/Tick #219

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

crisman
Copy link
Contributor

@crisman crisman commented Mar 24, 2023

On the tour slide "tour/concurrency/6" ("Default Selection") the example code in "default-selection.go" uses time.After() and time.Tick() without any mention of what they do. A link to the time fuctions doc can provide context to understand what those do.

Add link to time#pkg-functions on slide next to code using time.After()/Tick().

Fixes golang/tour#1466

On the tour slide "tour/concurrency/6" ("Default Selection") the example
code in "default-selection.go" uses time.After() and time.Tick() without
any mention of what they do. A link to the time fuctions doc can provide
context to understand what those do.

Add link to time#pkg-functions on slide next to code using
time.After()/Tick().

Fixes golang/tour#1466
@gopherbot
Copy link
Contributor

This PR (HEAD: 655fec7) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/website/+/479155 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants