Skip to content

test,doc: correct documentation for runBenchmark() - #40683

Closed
Trott wants to merge 0 commit into
nodejs:masterfrom
Trott:runBenchmark
Closed

test,doc: correct documentation for runBenchmark()#40683
Trott wants to merge 0 commit into
nodejs:masterfrom
Trott:runBenchmark

Conversation

@Trott

@TrottTrott commented Nov 1, 2021

Copy link
Copy Markdown
Member

The function only takes two arguments.

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Nov 1, 2021
@targos

Copy link
Copy Markdown
Member

s/Bemnchmark/Benchmark/

@TrottTrott changed the title test,doc: correct documentation for runBemnchmark()test,doc: correct documentation for runBenchmark()Nov 1, 2021
@TrottTrott added the fast-track PRs that do not need to wait for 48 hours to land. label Nov 1, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @Trott. Please 👍 to approve.

@TrottTrott added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 1, 2021
@github-actionsgithub-actionsBot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Nov 1, 2021
@github-actions

Copy link
Copy Markdown
Contributor
Commit Queue failed
- Loading data for nodejs/node/pull/40683
✔ Done loading data for nodejs/node/pull/40683
----------------------------------- PR info ------------------------------------
Title test,doc: correct documentation for runBenchmark() (#40683)
Author Rich Trott (@Trott)
Branch Trott:runBenchmark -> nodejs:master
Labels test, fast-track, needs-ci
Commits 1
- test,doc: correct documentation for runBenchmark()
Committers 1
- Rich Trott PR-URL: https://github.com/nodejs/node/pull/40683
Reviewed-By: Michaël Zasso Reviewed-By: Tobias Nießen Reviewed-By: Richard Lau Reviewed-By: Voltrex Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40683
Reviewed-By: Michaël Zasso Reviewed-By: Tobias Nießen Reviewed-By: Richard Lau Reviewed-By: Voltrex Reviewed-By: Luigi Pinca --------------------------------------------------------------------------------
⚠ Commits were pushed since the last review:
⚠ - test,doc: correct documentation for runBenchmark()
ℹ This PR was created on Mon, 01 Nov 2021 04:39:49 GMT
✔ Approvals: 5
✔ - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/40683#pullrequestreview-793889485
✔ - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/40683#pullrequestreview-794110285
✔ - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/40683#pullrequestreview-794112267
✔ - Voltrex (@VoltrexMaster): https://github.com/nodejs/node/pull/40683#pullrequestreview-794320002
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/40683#pullrequestreview-794618497
ℹ This PR is being fast-tracked
✔ Last GitHub Actions successful
✖ No Jenkins CI runs detected
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1409997070

@TrottTrott removed the needs-ci PRs that need a full CI run. label Nov 1, 2021
Trott added a commit to Trott/io.js that referenced this pull request Nov 1, 2021
The function only takes two arguments.
PR-URL: nodejs#40683
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@TrottTrott closed this Nov 1, 2021
@Trott

Trott commented Nov 1, 2021

Copy link
Copy Markdown
MemberAuthor

Landed in f867b61

@Trott
Trott deleted the runBenchmark branch November 1, 2021 22:55
targos pushed a commit that referenced this pull request Nov 6, 2021
The function only takes two arguments.
PR-URL: #40683
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Nov 8, 2021
BethGriggs pushed a commit that referenced this pull request Nov 25, 2021
The function only takes two arguments.
PR-URL: #40683
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Nov 26, 2021
1 task
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-failedAn error occurred while landing this pull request using GitHub Actions.fast-trackPRs that do not need to wait for 48 hours to land.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@Trott@targos@lpinca@tniessen@richardlau@VoltrexKeyva@nodejs-github-bot