Skip to content

First try locally before recursive step - #2726

Merged
KevinRansom merged 2 commits into
dotnet:masterfrom
forki:ValInfos2
Mar 31, 2017
Merged

First try locally before recursive step#2726
KevinRansom merged 2 commits into
dotnet:masterfrom
forki:ValInfos2

Conversation

@forki

Copy link
Copy Markdown
Contributor

No description provided.

@forki
forkiforce-pushed the ValInfos2 branch 2 times, most recently from b5a7003 to 91e75efCompareMarch 29, 2017 11:53
@smoothdeveloper

Copy link
Copy Markdown
Contributor

@forki what is the background for this change and do you know where this is used / what this code is for?

This looks good.

@forki

Copy link
Copy Markdown
ContributorAuthor

@smoothdeveloper I'm reading the code and wondered. This is more for discussion ;-)

@dsyme

Copy link
Copy Markdown
Contributor

@forki Is that function used anywhere?

@forki

Copy link
Copy Markdown
ContributorAuthor

lol good catch @dsyme - I removed the thing

@KevinRansom

Copy link
Copy Markdown
Contributor

Nice ... thanks for this mate

Kevin

@KevinRansom
KevinRansom merged commit e9ec864 into dotnet:masterMar 31, 2017
Sign up for freeto 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

@forki@smoothdeveloper@dsyme@KevinRansom@msftclas