[Improvement] Better error message when in detached head

[Evangelink]

Is your improvement request related to a problem? Please describe.

When re-building a commit in history, I often fail to remember to comment the GitVersion.MsBuild package reference from my project. Which results in one failure per project with the following message:

Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB3073	The command ""C:\Users\XXX\.nuget\packages\gitversion.msbuild\5.6.10\tools\net48/gitversion.exe" "C:\Src\YYY" -output file -outputfile obj\gitversion.json" exited with code 1.	YYY	C:\Users\XXX\.nuget\packages\gitversion.msbuild\5.6.10\tools\GitVersion.MsBuild.targets	9	

This error message is pretty generic and could (maybe?) be improved for the case of detached head with a more helpful message.

Detailed Description

Either return a different exit code and document it and/or fail with a message that would be displayed. I haven't done msbuild tasks for quite some time and so I don't really remember what's possible and how to do so.

Context

Relates to #2697

Possible Implementation

[asbjornu]

Improvements in error messages are always welcome! A PR would be highly appreciated. 🙏🏼

[Evangelink]

I will try to provide something but if you have some spare time for a couple of advices, they would be welcome :)

[asbjornu]

If you are able to reproduce the problem with the command line, we can see whether the exception thrown is something sensible and start from there.

[Evangelink]

Just for the sake of grouping the info together, here are the steps to reproduce the issue:

• Install GitVersion on a project
• Make some commit
• Checkout previous commit
• Rebuild solution
• Build fail with error saying gitversion.exe failed

[stale]

This issue has been automatically marked as stale because it has not had recent activity. After 30 days from now, it will be closed if no further activity occurs. Thank you for your contributions.

[asbjornu]

A PR that improves the error message of detached heads is still very welcome. :)

[Evangelink]

Hey @asbjornu! I am really sorry about all that delay. I had a pretty busy end of year, I will try to give it a go during the week-end.

[asbjornu]

No problem, @Evangelink. Take we're in no rush. :)

[asbjornu]

Fixed by #3338.