Git version calculating wrong version when building a tag

[gerwinjansen]

Note: this PR is on top of #2411, that's why you see the changes again.

Description

TaggedCommitVersionStrategy will try to return the tags on the current commit. If there are none, it will returns the tags from the history.

Related Issue

Fixes #2074 GitVersion calculating wrong version when building a tag

How Has This Been Tested?

See steps to reproduce in my comments of the related ticket.

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[asbjornu]

This is great! But there's something odd going on with the Docker tests. Can you please look into that?

========================================
Docker-Test
========================================
Executing task: Docker-Test
Testing image: gittools/gitversion:5.4.0-linux-alpine.3.10-x64-netcoreapp2.1
Output : 5.4.0-PullRequest2413.140
An error occurred when executing task 'Docker-Test'.

----------------------------------------
Teardown
----------------------------------------
Executing custom teardown action...
Starting Teardown...
Repository info : IsMainRepo True, IsMainBranch False, IsTagged: False, IsPullRequest: True
Finished running tasks.
Error: One or more errors occurred. (Assert.Equal() Failure
                                 ↓ (pos 23)
Expected: ···.0-PullRequest2413.171
Actual:   ···.0-PullRequest2413.140
                                 ↑ (pos 23))
	Assert.Equal() Failure
                                 ↓ (pos 23)
Expected: ···.0-PullRequest2413.171
Actual:   ···.0-PullRequest2413.140
                                 ↑ (pos 23)
Error: Process completed with exit code 1.

[gerwinjansen]

I have no clue how to run the docker tests locally and debug them.
Can you help me out?

[asbjornu]

Can you provide some guidance, @arturcic?

[asbjornu]

Can you please rebase on the HEAD of master and resolve conflicts, @gerwinjansen?

[gerwinjansen]

@asbjornu: build is succesful (without any changes from my side). Can this PR be completed now?

[asbjornu]

Excellent, thank you so much for your contribution @gerwinjansen! 🙏