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 GitLab support #658
Add GitLab support #658
Conversation
Add the possibility to add plugins from gitlab. Resolves jorgebucaran#657.
Even though this did work, it's not correct. This makes the URL fully correct.
README.md
Outdated
To install a plugin from GitLab, prefix `gitlab.com/`. | ||
|
||
```console | ||
fisher install gitlab.com/ainmosni/fish-aws | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ainmosni If you can turn this into one sentence and put it after a >
(we don't need the example), then I'd be happy to include this in the README.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you. I'll think a bit more about it.
Make it a bit less verbose.
if string match -q 'gitlab.com/*' \$name[1] | ||
set bare (string replace 'gitlab.com/' '' \$name[1]) | ||
set repo (string split '/' \$bare) | ||
set url https://gitlab.com/{\$bare}/-/archive/{\$name[2]}/{\$repo[2]}-{\$name[2]}.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It won't work with project in sub-group on Gitlab.
Like gitlab.com/config/fish/abc.git
project.
You must change it to
set url https://gitlab.com/{\$bare}/-/archive/{\$name[2]}/{\$repo[-1]}-{\$name[2]}.tar.gz
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for catching this bug. 💯
I'd really like to have this, did your thinking bring something up? 😋 |
Available in 4.3.5. Thank you, @ainmosni. 🙌 |
Add the possibility to add plugins from gitlab. Resolves #657.