Skip to content

n-api,test: remove superfluous persistent - #20299

Closed
gabrielschulhof wants to merge 1 commit into
nodejs:masterfrom
gabrielschulhof:n-api-clean-up-constructor-test
Closed

n-api,test: remove superfluous persistent#20299
gabrielschulhof wants to merge 1 commit into
nodejs:masterfrom
gabrielschulhof:n-api-clean-up-constructor-test

Conversation

@gabrielschulhof

Copy link
Copy Markdown
Contributor

Remove a superfluos persistent from test_constructor_name.c.

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

Remove a superfluos persistent from test_constructor_name.c.
@nodejs-github-bot nodejs-github-bot added dont-land-on-v4.x node-api Issues and PRs related to Node-API. test Issues and PRs related to Node.js core tests and test infrastructure. labels Apr 25, 2018
@BridgeAR

Copy link
Copy Markdown
Member

@BridgeAR BridgeAR added author ready PRs with CI started, the required approvals, and no outstanding review comments. fast-track PRs proposed for a shorter-than-standard waiting period before landing. labels Apr 26, 2018
gabrielschulhof pushed a commit to gabrielschulhof/node that referenced this pull request Apr 26, 2018
Remove a superfluos persistent from test_constructor_name.c.

PR-URL: nodejs#20299
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@gabrielschulhof
gabrielschulhof deleted the n-api-clean-up-constructor-test branch April 26, 2018 15:19
MylesBorins pushed a commit that referenced this pull request May 4, 2018
Remove a superfluos persistent from test_constructor_name.c.

PR-URL: #20299
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@MylesBorins MylesBorins mentioned this pull request May 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. fast-track PRs proposed for a shorter-than-standard waiting period before landing. node-api Issues and PRs related to Node-API. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants