Update paperclip-vips to use kt-paperclip #3
Conversation
|
We have looked at swapping this over but unfortunately we have a lot of other dependencies that rely on the original gem file. Are you using this gem in production @todd-tsu ? |
| @@ -7,7 +7,7 @@ A custom paperclip processor that uses the [libvips image processing library](ht | |||
| Add this line to your application's Gemfile: | |||
|
|
|||
| ```ruby | |||
| gem 'paperclip-vips' | |||
| gem 'paperclip-vips', :git=>'https://github.com/tsu-social/paperclip-vips.git' | |||
There was a problem hiding this comment.
Wouldn't need this if merging
With paperclip deprecated, we quickly moved to kt-paperclip. I'm happy to create a new gem entirely for this that can pull from any changes if you think that's a better way to go, like kt-paperclip-vips.
Not yet. We are evaluating it for production use though. If we end up not using it, I wouldn't be the right choice to maintain any upstream changes though. |
|
@todd-tsu I think it makes sense to fork this and run with If you are still evaluating image libraries I highly recommend giving shrine a go - it is great and we are using it in our new projects. It is super customizable and has vips support out of the box. |
With the main paperclip branch being deprecated and unsupported, I think it would be beneficial to have paperclip-vips migrate to using the supported branch, kt-paperclip.