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

Add new daily job to clean cache in public/tmp (as suggested in #5946). #6147

Closed
wants to merge 2 commits into
base: develop
from

Conversation

Projects
None yet
5 participants
@cmrd-senya
Copy link
Member

cmrd-senya commented Jun 29, 2015

fixes #5946

def perform
CarrierWave.clean_cached_files!
end
end

This comment has been minimized.

@houndci-bot

houndci-bot Jun 29, 2015

Trailing whitespace detected.

sidekiq_options queue: :maintenance

recurrence { daily }

This comment has been minimized.

@houndci-bot

houndci-bot Jun 29, 2015

Trailing whitespace detected.

include Sidetiq::Schedulable

sidekiq_options queue: :maintenance

This comment has been minimized.

@houndci-bot

houndci-bot Jun 29, 2015

Trailing whitespace detected.

module Workers
class CleanCachedFiles < Base
include Sidetiq::Schedulable

This comment has been minimized.

@houndci-bot

houndci-bot Jun 29, 2015

Trailing whitespace detected.

end
end
end

This comment has been minimized.

@houndci-bot

houndci-bot Jun 29, 2015

1 trailing blank lines detected.

@denschub

This comment has been minimized.

Copy link
Member

denschub commented Jul 3, 2015

@cmrd-senya Do you mind fixing those whitespace errors raised by hound?

@cmrd-senya

This comment has been minimized.

Copy link
Member

cmrd-senya commented Jul 3, 2015

Sure I don't. Should I just make a new PR?

@Flaburgan

This comment has been minimized.

Copy link
Member

Flaburgan commented Jul 3, 2015

You can push to this branch, it will update this pull request.

@jhass

This comment has been minimized.

Copy link
Member

jhass commented Jul 3, 2015

Merged as fe3ff10

Thank you!

@jhass jhass closed this Jul 3, 2015

@jhass jhass added this to the 0.5.2.0 milestone Jul 3, 2015

@cmrd-senya cmrd-senya deleted the cmrd-senya:develop branch Jul 8, 2015

@KentShikama KentShikama referenced this pull request Jul 11, 2015

Merged

[WIP] OpenID Connect #6095

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment