Skip to content

Clone 1000 commits on Travis CI - #2124

Merged
Travis Plunk (TravisEz13) merged 1 commit into
PowerShell:masterfrom
andyleejordan:shallow
Aug 30, 2016
Merged

Travis Plunk (TravisEz13) merged 1 commit into
PowerShell:masterfrom
andyleejordan:shallow

Conversation

@andyleejordan

Copy link
Copy Markdown
Member

The previous failure we saw was because the most recent annotated tag did not exist in the previous 50 commits. In my experience we've gone about ~300 commits between tags, so I believe 1000 to be safe, but know 100 will not be.

We should continue to use git describe because it encodes both the most recent version and the exact commit past it where it's built.

Resolves #2029.

Finding out right now from Travis CI if it's possible to completely disable shallow cloning.

The previous failure we saw was because the most recent annotated tag
did not exist in the previous 50 commits. In my experience we've gone
about ~300 commits between tags, so I believe 1000 to be safe, but know
100 will not be.
@msftclas

Hi Andy Jordan (@andschwa), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Andy Schwartzmeyer). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@TravisEz13
Travis Plunk (TravisEz13) merged commit 1a8b4e9 into PowerShell:master Aug 30, 2016
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026
The previous failure we saw was because the most recent annotated tag
did not exist in the previous 50 commits. In my experience we've gone
about ~300 commits between tags, so I believe 1000 to be safe, but know
100 will not be.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants