Consider moving images to a separate branch? #234
Replies: 5 comments
-
Hi Ilan! Sure I'm down to optimize the install speed of the plugin. Thanks for the idea of using other branches. Is this the conventional way of doing it? I have noticed that some repos are able to utilize some sort of GitHub CDN where if you click on the image it opens up by itself without the normal GitHub UI container around it. Did you also explore that option? Btw congrats on the new tide releases! |
Beta Was this translation helpful? Give feedback.
-
I mean, no, not as far as I know, but that's because most repos deliver released products like .exes etc. with anything unneeded stripped out. In contrast, Fisher literally just downloads a zipped folder of the branch. So the best way of doing this would probably be to have a seperate
That's an even less clean solution to the problem. For example, with an assets branch, I can do |
Beta Was this translation helpful? Give feedback.
-
Just as an example, powerlevel10k does this with a separate branch https://github.com/romkatv/powerlevel10k-media. It's especially important there, because he has a ton of images, as well as a few largish font files. I just think having a separate branch is cleaner, because it makes working with the repository more cohesive. |
Beta Was this translation helpful? Give feedback.
-
Ah I see this technique has been going since at least 2016 (source: https://gist.github.com/joncardasis/e6494afd538a400722545163eb2e1fa5) and is quite popular. I shall do it! I looked around looking for other techniques and I found this one: |
Beta Was this translation helpful? Give feedback.
-
Done in 8d877a9. |
Beta Was this translation helpful? Give feedback.
-
Hey Patrick!
I was wondering if you'd be open to the idea of moving readme images from the release branch to a separate branch like Tide does.
The primary advantage here is that download times are faster. Right now, fzf.fish takes ~3s to
fisher install
on my machine, wheras Tide takes about half a second. And since fisher downloads things asynchronously, fzf.fish is likely to always be holding up the horses 😄 It also just seems a bit silly to waste time and bandwidth on useless images.I know this is an incredibly small and nitpicky thing, but I think improving the user experience, even by a small amount, is worth it.
Beta Was this translation helpful? Give feedback.
All reactions