Dynamic repo should require a branch specified

[JakeGinnivan]

See #351 (comment)

When using the https://github.com/ParticularLabs/GitVersion/wiki/Feature:-Dynamically-Obtained-Repository feature (/url) we should throw if the /b argument is not used. The exception should contain information from that wiki page to make it easy for people to know what is going wrong.

[RaphHaddad]

I will pick this up and send a PR soon.

Ideally I would like a discussion around the PR before it's merged. I will put my concerns on the PR.

[JakeGinnivan]

Oops :P didn't read before I merged

[JakeGinnivan]

What were the concerns? Or ended up being ok?

[RaphHaddad]

They were more around the fact that we're throwing the base class
'Exception' everywhere. Is there any plan to have our own derived
Exceptions?

On 15 July 2015 at 23:00, Jake Ginnivan notifications@github.com wrote:

What were the concerns? Or ended up being ok?

—
Reply to this email directly or view it on GitHub
#367 (comment)
.

[JakeGinnivan]

The exception handling isn't great. No plan, but no objections to fix it up

[RaphHaddad]

Excellent. I will create an issue for this so that I don't forget. I will
get on this in the next few days.

On 16 July 2015 at 00:10, Jake Ginnivan notifications@github.com wrote:

The exception handling isn't great. No plan, but no objections to fix it up

—
Reply to this email directly or view it on GitHub
#367 (comment)
.