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

Add submodules feature to git provider #218

Merged
merged 1 commit into from
Jan 19, 2015
Merged

Conversation

dduvnjak
Copy link

This PR adds the ability to skip initializing git submodules when cloning a repository (i.e. git submodule update --init --recursive)
The default behaviour is still the same.

@@ -479,6 +488,7 @@ The vcsrepo module is slightly unusual in that it is simply a type and providers
* `ssh_identity` - The provider supports a configurable SSH identity file. (Available with `git` and `hg`.)
* `user` - The provider can run as a different user. (Available with `git`, `hg` and `cvs`.)
* `p4config` - The provider support setting the P4CONFIG environment. (Available with `p4`.)
* `submodules` - The provider supports repository submodules which can be optionally initiliazed. (Available with `git`.)

Choose a reason for hiding this comment

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

initialized

@sodabrew
Copy link

Thanks for your interest in the module! I think this is a useful option to add. +1 from me. Let's give a couple days to see if there are other comments or objections.

@pgib
Copy link

pgib commented Jan 16, 2015

+1 from me.

@sodabrew
Copy link

@dduvnjak Could you squash the typo fix commit into the main commit?

@dduvnjak
Copy link
Author

@sodabrew done.

sodabrew added a commit that referenced this pull request Jan 19, 2015
Add submodules feature to git provider
@sodabrew sodabrew merged commit d08d16b into puppetlabs:master Jan 19, 2015
cegeka-jenkins pushed a commit to cegeka/puppet-vcsrepo that referenced this pull request Jan 3, 2018
Add submodules feature to git provider
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants