capistrano / rails

Official Ruby on Rails specific tasks for Capistrano
http://www.capistranorb.com/
MIT License
867 stars 270 forks source link

Asset cleanup #251

Closed chubchenko closed 1 year ago

chubchenko commented 3 years ago

First of all thanks for your work :tada:

What do u think about specifying a default value for a keep_assets value based on the keep_releases value?

I'm wondering about it because I have performed a cap doctor and the keep_assets setting wasn't recognized by Capistrano 😞

JasonPoll commented 1 year ago

@chubchenko - that is exactly what I did, even with cap doctor saying :keep_assets isn't a recognized command, it does still have an effect during the deploy.

I just went with this in my deploy.rb:

set :keep_assets, -> { fetch(:keep_releases, 5) }
chubchenko commented 1 year ago

@JasonPoll Thank you! I did similar to what you suggested, but my point was that after you assign a value for "keep assets" it will still display it as not recognized.

chubchenko commented 1 year ago

Need to assign a default value, then Capistrano will recognize it.