Skip to content

test: really run addon tests on make test - #7542

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:really-run-addon-tests
Closed

test: really run addon tests on make test#7542
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:really-run-addon-tests

Conversation

@addaleax

@addaleaxaddaleax commented Jul 5, 2016

Copy link
Copy Markdown
Member

Fix a s/addon/addons/ typo in the Makefile.

CI: https://ci.nodejs.org/job/node-test-commit/3974/

Fix a `s/addon/addons/` typo in the Makefile.
@addaleaxaddaleax added the test Issues and PRs related to the tests. label Jul 5, 2016
@nodejs-github-botnodejs-github-bot added the build Issues and PRs related to build files or the CI. label Jul 5, 2016
@cjihrig

Copy link
Copy Markdown
Contributor

LGTM

1 similar comment
@jasnell

Copy link
Copy Markdown
Member

LGTM

@bnoordhuis

Copy link
Copy Markdown
Member

Ai, did I really misspell that? LGTM.

@addaleax

Copy link
Copy Markdown
MemberAuthor

Landed in 1fe0708

@addaleaxaddaleax closed this Jul 5, 2016
@addaleax
addaleax deleted the really-run-addon-tests branch July 5, 2016 21:09
addaleax added a commit that referenced this pull request Jul 5, 2016
Fix a `s/addon/addons/` typo in the Makefile.
PR-URL: #7542
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@Fishrock123

Copy link
Copy Markdown
Contributor

OH

good catch!

Fishrock123 pushed a commit that referenced this pull request Jul 6, 2016
Fix a `s/addon/addons/` typo in the Makefile.
PR-URL: #7542
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@Fishrock123Fishrock123 mentioned this pull request Jul 6, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@addaleax@cjihrig@jasnell@bnoordhuis@Fishrock123@nodejs-github-bot