Added separate parameter for submodules fetch depth #392
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Using the same value of the fetch depth in the main repository and submodules can lead to a problem that cloned submodule will not contain commit referenced in the main repository. If the depth is not enough and there are commits in the submodule repository on top of the referenced one, the submodule update can fail with the following output:
At the same time increasing general fetch depth will increase fetch timing for the main repository.
The proposal is to separate the parameters by introduction of the 'submodules-fetch-depth' parameter that will default to
fetch-depth value if not present.