Failure after shallow clone via GitHub actions/checkout@v2

[gitfool]

See actions/checkout#113. I presume GitVersion can't do anything about this, except maybe improve the error message at non-diagnostic verbosity, although that may be more of a Cake build issue?

[StanleyGoldman]

This should be fixed by #1966

[StanleyGoldman]

You can test it out by using the artifacts from that pull request in the appveyor artifacts from that PR.
https://ci.appveyor.com/project/GitTools/gitversion/builds/29570366/artifacts

[gitfool]

@StanleyGoldman thanks for the heads up!

[gep13]

@gitfool from the looks of your build script:

https://github.com/gitfool/Cake.Dungeon/blob/master/scripts/version.cake#L6

You aren't calling GitVersion with the buildserver argument, so I don't think that the work @StanleyGoldman is doing will take effect until to call that argument.

[matilote]

GitVersion keeps failing on Github Actions due to

InvalidOperationException: Could not find a 'develop' or 'master' branch, neither locally nor remotely.

even after prior fetching all branches and tags.

It does work on pull requests without any errors.

[gitfool]

@gep13 I'm looking at this just now and not sure about using buildserver as I also need the GitVersion variables for all builds. I guess there's no harm in calling GitVersion twice, as done in Cake.Recipe, although I'm wondering why it reads the environment variables when it overwrites them soon after. 🤔

[gep13]

@gitfool running it twice shouldn't cause any problems. The first execution will cache the asserted version number into the local cache file, and the second execution will simply read from that file.

The Cake.Recipe logic could well be something that needs to be looked at again, it has gone through a number of iterations, and perhaps is now wrong.

[gitfool]

@gep13 FTR, that makes no difference and it looks like I still need to avoid a shallow clone.