allow fetch tags too

[bluelovers]

current it will add --no-tags

but i hope can fetch tag too

that is help when wanna build changelog base on tag in github actions

[koraktor]

Using fetch-depth: 0 will cause the refspec for tags to be added, effectively removing the --no-tags option:

checkout/src/git-command-manager.ts

Lines 175 to 177 in ec3a7ce

	if (!refSpec.some(x => x === refHelper.tagsRefSpec)) {
	args.push('--no-tags')
	}

Apparently, this might not be enough depending on your workflow. I‘m currently trying to implement this for koraktor/mavanagaiata which checks if the commit is tagged before releasing a new version. This check fails and I‘m still trying to find the cause for this.

[anthony-steele-cko]

It is correct that if you add fetch-depth: 0 then you will get tags.

But this leaves you with two choices:

• get no tags ( fetch-depth > 0 )
• get the whole history, everything back to the start of the repo ( fetch-depth = 0 )

If you need the latest tag, then the first won't work at all.
If the repo is large and old, the then second isn't great either.

We should be able to get a bounded number of recent commits and tags by specifying some options.

[anthony-steele-cko]

See also: #448

[anthony-steele-cko]

As a workaround, you can fetch tags as a step after the actions/checkout, as it leaves git in a valid state with credentials to do other operations. e.g.:

      - name: Checkout
        uses: actions/checkout@v3
      - name: Get tags
        run: git fetch --tags origin

As I understand it, this will fetch all tags but that's preferable to fetching all history when you want a tag.

[tiagovrtr]

This would be really useful so that things like dolfinus/setuptools-git-versioning would work in Actions

[mattpitkin]

@anthony-steele-cko Thanks for your suggested workaround. Unfortunately, I get the error:

git fetch --tags origin
  shell: sh -e {0}
fatal: unsafe repository ('/__w/cwinpy/cwinpy' is owned by someone else)
To add an exception for this directory, call:
	git config --global --add safe.directory /__w/cwinpy/cwinpy

despite git config --global --add safe.directory /__w/cwinpy/cwinpy being performed during the checkout action.

[alok87]

after git fetch --tags: git describe --tags --always --dirty is giving v1.0.0-beta.1-44-g898e851 response in my local when all tags are present but it gives tagless response in github action 🤔

had to do fetch-depth: 0

[markstijnman-shell]

Ran into this today as well. Turns out there is already a PR for this too: #579. Hopefully the maintainers can merge it soon, because I really don't fancy setting fetch_depth=0 on large, old repositories. Also, the link between fetch_depth and the fetching or not of tags is undocumented behavior, and I don't like having to rely on undocumented behavior.