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

doc: fix link on timers.md #8488

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
9 participants
@yorkie
Member

yorkie commented Sep 11, 2016

Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

Fix links on timers.md.

@Trott

This comment has been minimized.

Show comment
Hide comment
@Trott

Trott Sep 11, 2016

Member

LGTM

Member

Trott commented Sep 11, 2016

LGTM

@cjihrig

This comment has been minimized.

Show comment
Hide comment
@cjihrig

cjihrig Sep 11, 2016

Contributor

LGTM

Contributor

cjihrig commented Sep 11, 2016

LGTM

@lpinca

This comment has been minimized.

Show comment
Hide comment
@lpinca

lpinca Sep 11, 2016

Member

LGTM

Member

lpinca commented Sep 11, 2016

LGTM

@addaleax

This comment has been minimized.

Show comment
Hide comment
@addaleax

addaleax Sep 11, 2016

Member

LGTM

Member

addaleax commented Sep 11, 2016

LGTM

@TimothyGu

This comment has been minimized.

Show comment
Hide comment
@TimothyGu

TimothyGu Sep 11, 2016

Member

LGTM.

Member

TimothyGu commented Sep 11, 2016

LGTM.

@jasnell

This comment has been minimized.

Show comment
Hide comment
@jasnell

jasnell Sep 12, 2016

Member

LGTM

Member

jasnell commented Sep 12, 2016

LGTM

jasnell added a commit that referenced this pull request Sep 12, 2016

doc: fix link on timers.md
PR-URL: #8488
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell

This comment has been minimized.

Show comment
Hide comment
@jasnell

jasnell Sep 12, 2016

Member

Landed in ebaa69b

Member

jasnell commented Sep 12, 2016

Landed in ebaa69b

@jasnell jasnell closed this Sep 12, 2016

@yorkie yorkie deleted the yorkie:fix/timers-api-doc branch Sep 12, 2016

Fishrock123 added a commit that referenced this pull request Sep 14, 2016

doc: fix link on timers.md
PR-URL: #8488
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment