zdharma-continuum / zinit

🌻 Flexible and fast ZSH plugin manager

MIT License

3.2k stars 129 forks source link

fix: add `command` before instances of `git` that didn't have it #643

Closed kiliantyler closed 8 months ago

kiliantyler commented 8 months ago

Description

Motivation and Context

If git is aliased to something that doesn't work identically (e.g. bit) then zinit update <plugin> can fail, as well as zinit version

Most other instances of git already use command git, there was just a few missing.

How Has This Been Tested?

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation change
[ ] New feature (non-breaking change which adds functionality)

Checklist:

[ ] All new and existing tests passed.
[ ] I have added tests to cover my changes.
[ ] I have updated the documentation accordingly.

vladdoster commented 8 months ago

Did you check zinit{,-{additional,install,side}}.zsh or just zinit-autoload.zsh?

vladdoster commented 8 months ago

644 fixes failing test

vladdoster commented 8 months ago

@kiliantyler,

Thank you for taking the time to improve Zinit!