search

nesquena / rabl

General ruby templating with json, bson, xml, plist and msgpack support
http://blog.codepath.com/2011/06/27/building-a-platform-api-on-rails/
MIT License
3.64k stars 334 forks source link

Remove not required files from the .gem #737

Closed ggrocco closed 4 years ago

ggrocco commented 4 years ago

To make the .gem smaller and lighter remove the tests, fixtures, and examples.

At the fixture and tests has many symlinks and this makes very complicated on Open Source Code Scanner because these links broke after installing the gem, this way these files will not be at the production environments and make the gem more clear.

tagliala commented 4 years ago

Thanks! 👍

Also big improvement in file size

 36K    rabl-0.14.3.gem.ggrocco
164K    rabl-0.14.3.gem

@nesquena is this ok for you?

PS: is Rakefile needed in the gem?

ggrocco commented 4 years ago

Thanks!

Also big improvement in file size

 36K  rabl-0.14.3.gem.ggrocco
164K  rabl-0.14.3.gem

@nesquena is this ok for you?

PS: is Rakefile needed in the gem?

@tagliala based on the content on the Rakefile is only for build and test, I will remove too...

tagliala commented 4 years ago

I've also noticed a huge number of warnings related to symlinks gone.

There are others about open dependencies, but we can address those in a separate PR

tagliala commented 4 years ago

Thanks 👍

I've tested the new gem build locally and it works as expected

nesquena commented 4 years ago

Thanks!