Skip to content

doc: more detail on when timers are executed#7166

Closed
tristanls wants to merge 1 commit intonodejs:masterfrom
tristanls:master
Closed

doc: more detail on when timers are executed#7166
tristanls wants to merge 1 commit intonodejs:masterfrom
tristanls:master

Conversation

@tristanls
Copy link
Contributor

@tristanls tristanls commented Jun 5, 2016

Checklist
  • documentation is changed or added
  • the commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

This is an attempt to add information about when timers are executed in the event loop. I have no idea if this is correct. I've attempted to incorporate what I found in https://nodesource.com/blog/understanding-the-nodejs-event-loop/ and #7145.

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jun 5, 2016
@addaleax addaleax added the timers Issues and PRs related to the timers subsystem / setImmediate, setInterval, setTimeout. label Jun 5, 2016
@jasnell
Copy link
Member

jasnell commented Jun 6, 2016

Ref: #6937

I'd like to get #6937 landed before this.

@tristanls
Copy link
Contributor Author

tristanls commented Jun 6, 2016

I'll revisit this once #6937 lands.

@imyller
Copy link
Member

imyller commented Sep 15, 2016

#6937 has landed.

@TristanIS maybe it is time to revisit this PR?

@tristanls
Copy link
Contributor Author

tristanls commented Sep 16, 2016

Seems to me this PR is now obsolete.

@tristanls tristanls closed this Sep 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

doc Issues and PRs related to the documentations. timers Issues and PRs related to the timers subsystem / setImmediate, setInterval, setTimeout.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@tristanls @jasnell @imyller @addaleax @nodejs-github-bot