Add Update-VSTeamPullRequest and Add-VSTeamPullRequest

[MichelZ]

PR Summary

Add Update-VSTeamPullRequest
Add Add-VSTeamPullRequest
Fixes #120
Fixes #131

PR Checklist

• Write Help
• Write Unit Test
• Update CHANGELOG.md

[SebastianSchuetze]

thx, gonna review this soon

[MichelZ]

I think CI builds fail not because of the changes here, but because of flawed tests.
Fixed in #242

[DarqueWarrior]

OK I have a few questions on this one. In Update-VSTeamPullRequest:
Why do we need both -Draft and -Publish? It appears that -Draft should be enough. If it is present you set the body to true if not to false. Why do I need them both?

Also the ProjectName parameter is never used and seems like it can be removed from Update-VSTeamPullRequest.

I also don't think we need a Draft ParameterSet. Just remove that ParameterSet and remove all set names from Draft and Force.

[DarqueWarrior]

I will make the changes I think need to be made and commit them so we can review.

[DarqueWarrior]

Hey @MichelZ and @SebastianSchuetze can you look at my changes and make sure I have not missed anything.

[MichelZ]

@DarqueWarrior @SebastianSchuetze The reason that both are there: It's more explicit. e.g. if you update a draft to published, now you have to:
Update-VSTeamPullRequest -PullRequestId 1234
That doesn't seem descriptive or intuitive, where:
Update-VSTeamPullRequest -PullRequestId 1234 -Publish
feels very logical.

Also, if you want to e.g. update a draft pull request with anything, you have to think about including -Draft that it stays draft, else it gets automatically published. That screams "bug" to me :)