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: Move Hosting of Images #61

Open
dan9186 opened this issue Aug 20, 2017 · 2 comments
Open

Request: Move Hosting of Images #61

dan9186 opened this issue Aug 20, 2017 · 2 comments

Comments

@dan9186
Copy link
Contributor

dan9186 commented Aug 20, 2017

I'd first like to say thanks for all the efforts and time spent putting together such a useful project.

I use Goblin in almost every single Go project I ever write, and have noticed every time that I vendor my projects there is always the Goblin logo somewhere in that PR. It finally lead me to general curiosity of wondering how many instances of the goblin_logo.jpg and goblin_output.png are in my stuff.

I see that you've done a great job of reducing the size of those files, however it eventually add up. A look across my system right now with find $GOPATH/src -name "goblin_output.png" | wc -l says 78 of each. Napkin math says thats 4,251 KB of images vendored on my system right now.

If you were to move the images to a Github Pages repo like franela.github.io/goblin/images, and link it here that would be an easy win to remove the needless duplication of those images as Goblin is vendored across the Go landscape. All in all, I understand it's not terribly important and you may not wish to bother taking the time. I only wanted to point it out as a possibility to consider.

@xetorthio
Copy link
Contributor

xetorthio commented Aug 20, 2017 via email

@marcosnils
Copy link
Member

Makes totally sense. We'll move the images somewhere else.

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

No branches or pull requests

3 participants