openalm / Extension-UtilitiesPack

Release Management utility tasks
Other
34 stars 38 forks source link

Features/vsts new sdk port #50

Closed harshil93 closed 8 years ago

harshil93 commented 8 years ago

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

pascalberger commented 8 years ago

@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 commented 8 years ago

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.com<mailto: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/openalm/Extension-UtilitiesPack/pull/50#issuecomment-234790843, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABH30m-J5Adup4EAEr5cVYNQkU78tNfxks5qY6PhgaJpZM4JTnpv.

pascalberger commented 8 years ago

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