-
Notifications
You must be signed in to change notification settings - Fork 653
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
Shallow clone all submodules #1064
Conversation
4781cd4
to
aae76ef
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Don't merge this yet. The commit on master tests are broken. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we get this fixed before the next release?
506adfd
to
632be20
Compare
Looks like CI is erroring because of actions/checkout#424. |
b0c15ce
to
986e7e8
Compare
986e7e8
to
453f01e
Compare
453f01e
to
4c1d860
Compare
Don't shallow clone submodules (revert #1064)
This might improve clone times.
Type of change: new feature
Impact: software change
Release Notes
Switch to shallow clone of all submodules