Assorted typo fixes. by XhmikosR · Pull Request #1853 · nodejs/node-gyp · GitHub
Skip to content

Assorted typo fixes.#1853

Closed
XhmikosR wants to merge 1 commit into
nodejs:masterfrom
XhmikosR:master-xmr-typos
Closed

Assorted typo fixes.#1853
XhmikosR wants to merge 1 commit into
nodejs:masterfrom
XhmikosR:master-xmr-typos

Conversation

@XhmikosR

Copy link
Copy Markdown
Contributor
Checklist
  • npm install && npm test passes
  • tests are included
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

@cclauss cclauss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rvagg rvagg 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.

fine by me but can you see if your patch applies to http://github.com/refack/GYP and submit a PR there too please? we're about to vendor it in and blow away the changes made here.

@cclauss

cclauss commented Aug 12, 2019

Copy link
Copy Markdown
Contributor

rvagg pushed a commit that referenced this pull request Aug 20, 2019
PR-URL: #1853
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@rvagg

rvagg commented Aug 20, 2019

Copy link
Copy Markdown
Member

@rvagg rvagg closed this Aug 20, 2019
@XhmikosR XhmikosR deleted the master-xmr-typos branch August 20, 2019 07:21
@rvagg rvagg mentioned this pull request Sep 26, 2019
rvagg pushed a commit that referenced this pull request Sep 26, 2019
PR-URL: #1853
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@rvagg rvagg mentioned this pull request Sep 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants