Skip to content

test: improve assertion in test-performance - #20809

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:test-perf-assertion
Closed

test: improve assertion in test-performance#20809
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:test-perf-assertion

Conversation

@addaleax

Copy link
Copy Markdown
Member
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label May 17, 2018
@addaleaxaddaleax added perf_hooks Issues and PRs related to the implementation of the Performance Timing API. fast-track PRs that do not need to wait for 48 hours to land. labels May 17, 2018
@addaleax

Copy link
Copy Markdown
MemberAuthor

CI: https://ci.nodejs.org/job/node-test-pull-request/14940/

Feel free to 👍 this comment if you think fast-tracking might be nice.

@BridgeARBridgeAR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 17, 2018
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request May 18, 2018
PR-URL: nodejs#20809
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
@BridgeAR

Copy link
Copy Markdown
Member

Landed in 42a4a60 🎉

MylesBorins pushed a commit that referenced this pull request May 22, 2018
PR-URL: #20809
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
@addaleaxaddaleax mentioned this pull request May 22, 2018
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.fast-trackPRs that do not need to wait for 48 hours to land.perf_hooksIssues and PRs related to the implementation of the Performance Timing API.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@addaleax@BridgeAR@danbev@nodejs-github-bot