Skip to content

Revert "doc: correct vcbuild options for windows testing" - #10839

Merged
gibfahn merged 0 commit into
nodejs:masterfrom
gibfahn:revert-vcbuild-nosign
Jan 21, 2017
Merged

Revert "doc: correct vcbuild options for windows testing"#10839
gibfahn merged 0 commit into
nodejs:masterfrom
gibfahn:revert-vcbuild-nosign

Conversation

@gibfahn

Copy link
Copy Markdown
Member

This reverts commit ed9b6c1.

Basically #10686 shouldn't have landed in master, it was superseded by #10156. I've reopened #10686.

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

cc/ @MylesBorins@jasnell let me know if I'm doing this right, basically did a git revert ed9b6c1264a07084e44f1d269bfe82e0f1b37c74.

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. dont-land-on-v7.x labels Jan 16, 2017
@addaleax

Copy link
Copy Markdown
Member

@gibfahn You self-assigned this … should we wait with landing this?

@gibfahngibfahn closed this Jan 21, 2017
@gibfahn
gibfahnforce-pushed the revert-vcbuild-nosign branch from 940722e to 93c4820CompareJanuary 21, 2017 12:03
@gibfahn
gibfahn merged commit 93c4820 into nodejs:masterJan 21, 2017
@gibfahn

Copy link
Copy Markdown
MemberAuthor

Landed in 93c4820

@gibfahn
gibfahn deleted the revert-vcbuild-nosign branch January 21, 2017 12:04
@gibfahn

Copy link
Copy Markdown
MemberAuthor

@addaleax Thanks for reminding me! In general feel free to land stuff I've self-assigned, it's more to keep it on my list than anything else.

italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 25, 2017
This reverts commit ed9b6c1.
PR-URL: nodejs#10839
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 27, 2017
This reverts commit ed9b6c1.
PR-URL: nodejs#10839
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
@italoacasasitaloacasas mentioned this pull request Jan 29, 2017
@jasnelljasnell mentioned this pull request Apr 4, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@gibfahn@addaleax@jasnell@evanlucas@italoacasas@nodejs-github-bot