Fixing copy-paste issue #2237

Closed
wants to merge 1 commit into
from

Projects

None yet

7 participants

@tvsonar
Contributor
tvsonar commented Apr 24, 2015

No description provided.

@srivatsn
Contributor

👍 Thanks @tvsonar

@srivatsn
Contributor
@VSadov
Member
VSadov commented Apr 24, 2015

Is it possible to add a test to make sure the scenario does no regress?

@gafter
Member
gafter commented Apr 24, 2015

👍

@tvsonar
Contributor
tvsonar commented Apr 28, 2015

I'll probably not add the test. It seems to me that this method is reachable from SyntaxExtensions.NormalizeWhitespace(), but I couldn't find any tests for SyntaxExtensions.

Also, this fix has been already added to your master branch 4 days ago.

@rasoulian

Thanks @tvsonar 👍

@VSadov
Member
VSadov commented Apr 28, 2015

Indeed, the change is already in.

@VSadov
Member
VSadov commented Apr 28, 2015

I guess, I will close the PR, then.

@VSadov VSadov closed this Apr 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment