Skip to content

doc: add missing supported timer values in timers.enable() - #52969

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:240514_add_missing_supported_timer_values_in_timers
May 15, 2024
Merged

doc: add missing supported timer values in timers.enable()#52969
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
deokjinkim:240514_add_missing_supported_timer_values_in_timers

Conversation

@deokjinkim

Copy link
Copy Markdown
Contributor

Some timer values such as setImmediate and clearImmediate are missed. And milliseconds which is argument of timers.tick() is optional and default is 1.

Refs: #49534 (comment)

Some timer values such as `setImmediate` and `clearImmediate` are
missed. And `milliseconds` which is argument of `timers.tick()`
is optional and default is 1.
Refs: nodejs#49534 (comment)
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/test_runner

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. test_runner Issues and PRs related to the test runner subsystem. labels May 13, 2024
@atlowChemiatlowChemi added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels May 14, 2024
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 15, 2024
@nodejs-github-bot
nodejs-github-bot merged commit 139c26b into nodejs:mainMay 15, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 139c26b

targos pushed a commit that referenced this pull request May 21, 2024
Some timer values such as `setImmediate` and `clearImmediate` are
missed. And `milliseconds` which is argument of `timers.tick()`
is optional and default is 1.
Refs: #49534 (comment)
PR-URL: #52969
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
@RafaelGSSRafaelGSS mentioned this pull request Jun 7, 2024
eliphazbouye pushed a commit to eliphazbouye/node that referenced this pull request Jun 20, 2024
Some timer values such as `setImmediate` and `clearImmediate` are
missed. And `milliseconds` which is argument of `timers.tick()`
is optional and default is 1.
Refs: nodejs#49534 (comment)
PR-URL: nodejs#52969
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
marco-ippolito pushed a commit that referenced this pull request Jul 19, 2024
Some timer values such as `setImmediate` and `clearImmediate` are
missed. And `milliseconds` which is argument of `timers.tick()`
is optional and default is 1.
Refs: #49534 (comment)
PR-URL: #52969
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
@marco-ippolitomarco-ippolito mentioned this pull request Jul 19, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.docIssues and PRs related to the documentations.test_runnerIssues and PRs related to the test runner subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@deokjinkim@nodejs-github-bot@benjamingr@MoLow@atlowChemi