Skip to content

test: add test for missing dynamic instantiate hook - #21506

Closed
targos wants to merge 1 commit into
nodejs:masterfrom
targos:test-add-esm-missing-dyn
Closed

test: add test for missing dynamic instantiate hook#21506
targos wants to merge 1 commit into
nodejs:masterfrom
targos:test-add-esm-missing-dyn

Conversation

@targos

@targostargos commented Jun 24, 2018

Copy link
Copy Markdown
Member

Depends on:
#21352
#21493

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

@targostargos added test Issues and PRs related to the tests. blocked PRs that are blocked by other issues or PRs. esm Issues and PRs related to the ECMAScript Modules implementation. labels Jun 24, 2018
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Jun 24, 2018
@targostargos added the wip Issues and PRs that are still a work in progress. label Jun 24, 2018

@benjamingrbenjamingr 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 given the other two land

@targos
targosforce-pushed the test-add-esm-missing-dyn branch from 6116551 to 543142fCompareJune 30, 2018 21:57
@targos

targos commented Jun 30, 2018

Copy link
Copy Markdown
MemberAuthor

@targos

Copy link
Copy Markdown
MemberAuthor

Landed in 908518d

@targostargos closed this Jul 3, 2018
@targos
targos deleted the test-add-esm-missing-dyn branch July 3, 2018 15:34
@targostargos removed wip Issues and PRs that are still a work in progress. blocked PRs that are blocked by other issues or PRs. labels Jul 3, 2018
targos added a commit that referenced this pull request Jul 3, 2018
PR-URL: #21506
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
targos added a commit that referenced this pull request Jul 4, 2018
PR-URL: #21506
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
@targostargos mentioned this pull request Jul 17, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

esmIssues and PRs related to the ECMAScript Modules implementation.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@targos@nodejs-github-bot@jasnell@benjamingr@devsnek