Skip to content

actions: Try checkout v2.2.0 #3035

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 15, 2020
Merged

Conversation

jepler
Copy link

@jepler jepler commented Jun 15, 2020

This version of actions/checkout will

Fetch all history for all tags and branches when fetch-depth=0

We leave the tags fetch in place so that actions in cloned repos work. Cloned repos' tags do not necessarily match adafruit/circuitpython, but we want version reporting to be able to use adafruit/circuitpython tags when they are most relevant according to "git describe"'s heuristics.

Submodules are different, as they always point to the repo specified in .gitmodules, so they don't need special handling to get the most relevant tags.

This version is supposed to
> Fetch all history for all tags and branches when fetch-depth=0

We leave the tags fetch in place so that actions _in cloned repos_
work.  Cloned repos' tags do not necessarily match adafruit/circuitpython,
but we want version reporting to be able to use adafruit/circuitpython
tags when they are most relevant according to "git describe"'s heuristics.

Submodules are different, as they always point to the repo specified
in .gitmodules, so they don't need special handling to get the most
relevant tags.
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@tannewt tannewt merged commit b3b38c1 into adafruit:main Jun 15, 2020
@jepler jepler deleted the simplify-checkout branch November 3, 2021 21:10
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.

2 participants