Skip to content

Conversation

@IvanWei
Copy link
Contributor

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

@nodejs-github-botnodejs-github-bot added the tools Issues and PRs related to the tools directory. label Nov 22, 2017
@MylesBorinsMylesBorins added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Nov 22, 2017
@joyeecheung
Copy link
Member

I am kind of wondering why are those remark-* not under tools/doc/node_modules? @watilde

@MylesBorins
Copy link
Contributor

Landed in c2c4dd4

Congrats on your first PR on core 🎉🎉🎉

MylesBorins pushed a commit that referenced this pull request Nov 23, 2017
PR-URL: #17240 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
@IvanWei
Copy link
ContributorAuthor

@MylesBorins

This is my second PR on core. 😄
( First PR: #17227)

I'm happy that you're coming Taiwan.
But my English is weak, I don't know how could I tell you that I'm excited last night. 😢

@IvanWeiIvanWei deleted the featrue/add_new_file_to_eslintignore branch November 27, 2017 04:20
MylesBorins pushed a commit that referenced this pull request Dec 12, 2017
PR-URL: #17240 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this pull request Dec 12, 2017
PR-URL: #17240 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Dec 12, 2017
gibfahn pushed a commit that referenced this pull request Dec 19, 2017
PR-URL: #17240 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
@gibfahngibfahn mentioned this pull request Dec 20, 2017
gibfahn pushed a commit that referenced this pull request Dec 20, 2017
PR-URL: #17240 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
@gibfahngibfahn mentioned this pull request Dec 20, 2017
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.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@IvanWei@joyeecheung@MylesBorins@jasnell@benjamingr@cjihrig@gibfahn@nodejs-github-bot