search

jorgebucaran / fisher

A plugin manager for Fish
https://git.io/fisher
MIT License
7.53k stars 257 forks source link

Updated README.md to copy the default install of tide #710

Closed AucaCoyan closed 2 years ago

AucaCoyan commented 2 years ago

I changed the tide install for the current install of tide in their readme: before:

fisher install ilancosman/tide

after

fisher install IlanCosman/tide@v5
jorgebucaran commented 2 years ago

Care to explain why?

AucaCoyan commented 2 years ago

Sorry for the delay! I wanted to fix the readme for updated installation command. I believe it is probable that the same user that installs fisher will also instal tide, so, to ease the path, I would copy and paste the same command of tide. Is that right?

eitsupi commented 2 years ago

The default branch of tide is for development, so if we install from the default branch we will get a warning to install v5 (I have experienced this)

jorgebucaran commented 2 years ago

Thank you, @eitsupi , I think that's the piece of info I was missing.

jorgebucaran commented 2 years ago

We'll need to move Tide to the second installation example, which is where we explain the use of @ to install tags or branches.

AucaCoyan commented 2 years ago

Thanks! You are right, I didn't explained myself correctly. I will take this as a lesson! 😄 . l feel satisfied to close the PR. Have a nice day!