Skip to content

test: updated to const and strictEqual - #9974

Closed
ghvaldez wants to merge 1 commit into
nodejs:masterfrom
ghvaldez:greg-valdez-update-test
Closed

test: updated to const and strictEqual#9974
ghvaldez wants to merge 1 commit into
nodejs:masterfrom
ghvaldez:greg-valdez-update-test

Conversation

@ghvaldez

Copy link
Copy Markdown
Contributor
Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

Tests

Description of change

Upadted all var to const/let and updated assert.equal to
assert.strictEqual

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Dec 1, 2016
@mscdexmscdex added the crypto Issues and PRs related to the crypto subsystem. label Dec 1, 2016
@imyllerimyller added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Dec 1, 2016
@mscdex

Copy link
Copy Markdown
Contributor

tests: should be test: in the commit message.

Upadted all var to const/let and updated assert.equal to
assert.strictEqual
@ghvaldez
ghvaldezforce-pushed the greg-valdez-update-test branch from 8d3c3ed to f3e3d79CompareDecember 1, 2016 17:08
@ghvaldez

Copy link
Copy Markdown
ContributorAuthor

@mscdex updated commit message

@Trott

Trott commented Dec 4, 2016

Copy link
Copy Markdown
Member

@TrottTrott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Trott pushed a commit to Trott/io.js that referenced this pull request Dec 4, 2016
Updated all remaining var to const
PR-URL: nodejs#9974
Reviewed-By: Rich Trott <rtrott@gmail.com>
@Trott

Trott commented Dec 4, 2016

Copy link
Copy Markdown
Member

Landed in 7837866. Thanks for the contribution! 🎉

@TrottTrott closed this Dec 4, 2016
addaleax pushed a commit that referenced this pull request Dec 5, 2016
Updated all remaining var to const
PR-URL: #9974
Reviewed-By: Rich Trott <rtrott@gmail.com>
@Fishrock123Fishrock123 mentioned this pull request Dec 5, 2016
2 tasks
addaleax pushed a commit to addaleax/node that referenced this pull request Dec 8, 2016
Updated all remaining var to const
PR-URL: nodejs#9974
Reviewed-By: Rich Trott <rtrott@gmail.com>
jmdarling pushed a commit to jmdarling/node that referenced this pull request Dec 8, 2016
Updated all remaining var to const
PR-URL: nodejs#9974
Reviewed-By: Rich Trott <rtrott@gmail.com>
MylesBorins pushed a commit that referenced this pull request Dec 21, 2016
Updated all remaining var to const
PR-URL: #9974
Reviewed-By: Rich Trott <rtrott@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Dec 21, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.cryptoIssues and PRs related to the crypto subsystem.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@ghvaldez@mscdex@Trott@MylesBorins@addaleax@imyller@nodejs-github-bot