Features/vsts new sdk port

[harshil93]

Porting the the task to use the new sdk and adding some tests

[pascalberger]

@harshil93 Shouldn't the old PS handler be removed? Or do you plan to maintain two versions. Sounds like too much trouble to me to keep them in sync.

[harshil93]

I will remove it after a month or so if no issues came up. People using TFS update 2 might not have the latest handler.

via CloudMagic Emailhttps://cloudmagic.com/k/d/mailapp?ct=pa&cv=8.5.49&pv=6.0.1&source=email_footer_2
On Sun, Jul 24, 2016 at 11:11 PM, Pascal Berger <notifications@github.commailto:notifications@github.com> wrote:

@harshil93https://github.com/harshil93 Shouldn't the old PS handler be removed? Or do you plan to maintain two versions. Sounds like too much trouble to me to keep them in sync.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com//pull/50#issuecomment-234790843, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABH30m-J5Adup4EAEr5cVYNQkU78tNfxks5qY6PhgaJpZM4JTnpv.

[pascalberger]

@harshil93 Any plans to merge this soon, as I'm waiting for this to fix #24?

[pascalberger]

This seems to be redundant to the Trace-VstsEnterinigInvocation call above. Also the output of the parameters