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

Shallow clone submodules by default #106

Merged
merged 5 commits into from
Nov 17, 2023

Conversation

6543
Copy link
Member

@6543 6543 commented Nov 15, 2023

got a pipeline (non public) from 12:35min down to 3:03min !!!


Sponsored by Kithara Software GmbH

@6543 6543 added the enhancement New feature or request label Nov 15, 2023
@6543 6543 modified the milestones: v3.0.0, v2.4.0 Nov 15, 2023
@6543 6543 requested a review from a team November 15, 2023 21:28
@6543
Copy link
Member Author

6543 commented Nov 15, 2023

Before

image

After

image

@pat-s
Copy link
Contributor

pat-s commented Nov 16, 2023

Should this behavior (and other custom defaults) be documented somewhere? I guess a high-level overview of how the default cloning is happening could go into https://github.com/woodpecker-ci/plugin-git/blob/main/docs.md and into https://woodpecker-ci.org/docs/next/usage/workflow-syntax#clone?

@6543
Copy link
Member Author

6543 commented Nov 17, 2023

we already document the defaults!!!

in docs.md

@6543
Copy link
Member Author

6543 commented Nov 17, 2023

image

@lafriks lafriks merged commit 74efdd5 into woodpecker-ci:main Nov 17, 2023
2 checks passed
@6543 6543 deleted the shallow-clone-submodules branch November 18, 2023 17:36
@woodpecker-bot woodpecker-bot mentioned this pull request Nov 21, 2023
1 task
@woodpecker-bot
Copy link
Contributor

🎉 This PR is included in version 2.4.0 🎉

The release is now available here

Thank you for your contribution. ❤️📦🚀

6543 added a commit to woodpecker-ci/woodpecker that referenced this pull request Nov 21, 2023
as with woodpecker-ci/plugin-git#106 we have a potential to speed up clone step a lot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants