search

railslove / fontello_rails_converter

CLI gem for comfortably working with icon fonts (open, download, convert) from http://fontello.com for usage in Rails apps.
MIT License
73 stars 35 forks source link

Recommend not loading into memory in production #31

Closed jaredbeck closed 9 years ago

jaredbeck commented 9 years ago

Per https://github.com/railslove/fontello_rails_converter/issues/15 we should be recommending that people not load this gem into memory in production, right? Every little bit of memory counts :)

jhilden commented 9 years ago

Thanks for the PR @jaredbeck

Since we already had a section in the README where to put the gem in your Gemfile, I just linked to that section instead of just recommending to put the gem into the development, because there are additional things people should be aware of. See 59869df4755949abd1814a723668ca996712501e

Does that work for you, or do you think it should be handeled differently?

jaredbeck commented 9 years ago

Wow. I didn't even notice the line one line below the line I changed... I'm bad at reading manuals :anguished:

Anyways, I guess it's a tradeoff between slightly more memory in production and slightly easier config? I can see why as a library author you'd rather recommend an easier config.