remove sidekiq

[taboularasa]

I suggest we remove sidekiq because it's easy to add to a project and not every project needs it

[patbenatar]

I originally added sidekiq to replace delayed_job which came in thoughtbot's version when we forked. I just did a quick assessment of recent projects to see how many use it:

• CSL: no
• Cowgirl: yes
• David Kind: yes
• CRM: yes
• Core Innovate: no
• Fliper: no
• VenMetro: yes

I vote we keep it even if not all projects need it (or just don't need it initially).. now that Rails has ActiveJob built in, we're just defaulting to a robust backend for it.

:-1:

[jdwolk]

I vote we keep in -- agree w/ @patbenatar that it makes sense as a robust default for ActiveJob

:thumbsdown:

[taboularasa]

:+1: to @jdwolk and @patbenatar :-1: