Skip to content
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

Bump checkout action version #3524

Closed
wants to merge 3 commits into from
Closed

Conversation

smorimoto
Copy link
Contributor

No description provided.

@claassistantio
Copy link

claassistantio commented Dec 19, 2019

CLA assistant check
All committers have signed the CLA.

@smorimoto
Copy link
Contributor Author

Submodule option is no longer supported. (and the default depth is 1, so we don't need that option.)

@ry
Copy link
Member

ry commented Dec 19, 2019

Is there some improvement? Checkout works fine and this looks more complex.

@smorimoto
Copy link
Contributor Author

Here is the changelog. It certainly looks a bit complicated, but I don't think it's a bad idea to keep following the new version. and current CI configuration is already a bit complicated.
https://github.com/actions/checkout/releases/tag/v2.0.0

@lucacasonato
Copy link
Member

Is there some improvement? Checkout works fine and this looks more complex.

Its apparently faster because it doesn't check out the entire history - only the commit that is actually being tested.

@hayd
Copy link
Contributor

hayd commented Jan 8, 2020

Best to wait til this is fixed in v2 (seems kinda crazy everyone would be copying this).
actions/checkout#81

@bartlomieju
Copy link
Member

Thanks @imbsky, but we're gonna stick with v1 for now

@smorimoto smorimoto deleted the patch-1 branch February 24, 2020 16:57
@smorimoto
Copy link
Contributor Author

2.1.0 has been released.
https://github.com/actions/checkout/releases/tag/v2.1.0

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.

6 participants