Removes trailing caret from git.tag

README.md

@@ -465,8 +465,9 @@ gitWithCreds 'https://your.repo' // Implicitly passed credentials
 * `git.commitHashShort` -  e.g. `fb1c882`
 * `git.repositoryUrl` -  e.g. `https://github.com/orga/repo.git`
 * `git.gitHubRepositoryName` -  e.g. `orga/repo`
-* `git.tag` -  e.g. `1.0.0` or `undefined` if not set
-* `git.isTag()` - is there a tag on the current commit?
+* Tags - Note that the git plugin might not fetch tags for all builds. Run `sh "git fetch --tags"` to make sure.
+    * `git.tag` -  e.g. `1.0.0` or `undefined` if not set
+    * `git.isTag()` - is there a tag on the current commit?
 
 ### Changes to local repository
 

src/com/cloudogu/ces/cesbuildlib/Git.groovy

@@ -128,11 +128,12 @@ class Git implements Serializable {
     }
 
     String getTag() {
-        return sh.returnStdOut("git name-rev --name-only --tags HEAD")
+        // Note that "git name-rev --name-only --tags HEAD" always seems to append a caret (e.g. "1.0.0^") 
+        return sh.returnStdOut("git tag --points-at HEAD")
     }
 
     boolean isTag() {
-        return getTag() != "undefined"
+        return !getTag().isEmpty()
     }
 
     def add(String pathspec) {

test/com/cloudogu/ces/cesbuildlib/GitTest.groovy

@@ -183,7 +183,7 @@ class GitTest {
 
     @Test
     void isNotATag() {
-        String expectedReturnValue = "undefined"
+        String expectedReturnValue = ""
         scriptMock.expectedDefaultShRetValue = expectedReturnValue + " \n"
         assertFalse(git.isTag())
     }