FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: replace common.fixturesDir with usage of the common.fixtures by aniwng · Pull Request #15803 · nodejs/node · GitHub

/ node Public

test: replace common.fixturesDir with usage of the common.fixtures - #15803

Closed
aniwng wants to merge 1 commit into
nodejs:masterfrom
aniwng:master
Closed

test: replace common.fixturesDir with usage of the common.fixtures#15803
aniwng wants to merge 1 commit into
nodejs:masterfrom
aniwng:master

Conversation

aniwng commented Oct 6, 2017

Copy link
Copy Markdown
Contributor

module

nodejs-github-bot added the test Issues and PRs related to the tests. label Oct 6, 2017
aniwng closed this Oct 6, 2017
Trott added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Oct 6, 2017

gibfahn commented Oct 7, 2017

Copy link
Copy Markdown
Member

@aniwng did you mean to close this? If not we should get this landed!

Code changes LGTM.

targos commented Oct 7, 2017

Copy link
Copy Markdown
Member

I think it was opened twice by mistake. See #15813

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL