Skip to content

Conversation

@acoates-ms
Copy link
Contributor

@acoates-ms acoates-ms commented Feb 20, 2020

Fixes #3611

Turns out we picked up the change from metro that was supposed to fix this, but we had duplicated the code issue in some of our local code, which this should fix.

Microsoft Reviewers: Open in CodeFlow

@acoates-ms acoates-ms requested a review from a team as a code owner February 20, 2020 18:23
@ghost ghost added the vnext label Feb 20, 2020
@acoates-ms acoates-ms added AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) and removed vnext labels Feb 20, 2020
@ghost
Copy link

ghost commented Feb 20, 2020

Hello @acoates-ms!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 60 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@acoates-ms acoates-ms removed the AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) label Feb 20, 2020
@NickGerleman
Copy link
Contributor

@acoates-ms could you also update the getting started docs that ask users to install Node 10?

Copy link
Member

@vmoroz vmoroz left a comment

Choose a reason for hiding this comment

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

:shipit:

@acoates-ms acoates-ms added the AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) label Feb 21, 2020
@ghost ghost merged commit 8945111 into master Feb 21, 2020
@ghost ghost deleted the node12 branch February 21, 2020 01:23
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate the integration with nodejs 12.13

4 participants