Skip to content

test: switch assertEqual arguments - #27912

Merged
ChALkeR merged 1 commit into
nodejs:masterfrom
zeckson:fix-assert-gh-892
May 28, 2019
Merged

test: switch assertEqual arguments#27912
ChALkeR merged 1 commit into
nodejs:masterfrom
zeckson:fix-assert-gh-892

Conversation

@zeckson

Copy link
Copy Markdown
Contributor
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 26, 2019
@ChALkeRChALkeR added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label May 26, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@ChALkeRChALkeR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 27, 2019
PR-URL: nodejs#27912
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@ChALkeR
ChALkeRforce-pushed the fix-assert-gh-892 branch from c151d50 to b810607CompareMay 28, 2019 20:43
@ChALkeR
ChALkeR merged commit b810607 into nodejs:masterMay 28, 2019
@ChALkeR

Copy link
Copy Markdown
Member

Landed in b810607, thanks for the contribution! 🎉

targos pushed a commit that referenced this pull request May 31, 2019
PR-URL: #27912
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Jun 3, 2019
@zeckson
zeckson deleted the fix-assert-gh-892 branch June 18, 2019 15:10
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.code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@zeckson@nodejs-github-bot@ChALkeR@Trott@addaleax@lpinca@cjihrig@gireeshpunathil