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

Request: new load syntax for _not_ cloning submodules #129

Open
AndydeCleyre opened this issue Aug 9, 2020 · 1 comment · Fixed by jandamm/zgenom#9
Open

Request: new load syntax for _not_ cloning submodules #129

AndydeCleyre opened this issue Aug 9, 2020 · 1 comment · Fixed by jandamm/zgenom#9

Comments

@AndydeCleyre
Copy link

A plugin/repo's submodules are not always needed for regular use, and in those cases, refraining from cloning them can save the user some space and bandwidth. A new syntax like zgen load --no-submodules... could be handy.

jandamm added a commit to jandamm/zgenom that referenced this issue Sep 21, 2020
In my opinion the `load` syntax is a bit overloaded already. Also adding
other commands like `bin` would have to implement this in multiple
places.
This can however be achieved like this:

```zsh
zgen clone <repo> --no-submodules
zgen load <repo>
```

Closes tarjoilija/zgen#129
@jandamm
Copy link

jandamm commented Sep 22, 2020

Hi, I've created a maintained fork of zgen - dubbed zgenom - which fixes some bugs as well as introducing new features.
Please let me know what you think of my solution.

jandamm added a commit to jandamm/zgenom that referenced this issue Oct 5, 2020
* Allow to clone without submodules

In my opinion the `load` syntax is a bit overloaded already. Also adding
other commands like `bin` would have to implement this in multiple
places.
This can however be achieved like this:

```zsh
zgen clone <repo> --no-submodules
zgen load <repo>
```

Closes tarjoilija/zgen#129

* Fix update initializing submodules

By adding `zgen clone ... --no-submodules` there is a way to not clone
submodules. So updating should not initialize submodules either.
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 a pull request may close this issue.

2 participants