search

toptal / active-job-style-guide

This Background Jobs style guide is a list of best practices working with Ruby background jobs.
471 stars 20 forks source link

Add section about 'perform_async vs perform_at or perform_in' #8

Closed ydakuka closed 1 year ago

ydakuka commented 1 year ago

References:

https://www.mikeperham.com/2021/11/07/whats-new-in-sidekiq-6.3/#activejob-compatibility

https://github.com/sidekiq/sidekiq/blob/main/Changes.md#630

pirj commented 1 year ago

I don’t find it convincing enough that if support for alternative syntax is added, is should immediately become the default. Not to say that one of those two should be preferred.

If some project strives for consistency, and has a strong preference for the style, maybe a custom cop would do, but I would not add this to the guide. Thanks for understanding.