Skip to content
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

Remove worker from Procfile #854

Conversation

ianagne
Copy link
Contributor

@ianagne ianagne commented May 3, 2017

The delayed_job_active_record gem was removed in #845 and the jobs:work
rake task is no longer available. Remove the worker setup from the
Procfile so rake jobs:work will not be called.

The delayed_job_active_record gem was removed in thoughtbot#845 and the jobs:work
rake task is no longer available.  Remove the worker setup from the
Procfile so rake jobs:work will not be called.
@ianagne
Copy link
Contributor Author

ianagne commented May 3, 2017

Of note, there's also a migration for the delayed_jobs table that should be removed. Would it be preferable to delete the migration outright, or should I add a drop_tables migration for posterity?

@nickcharlton
Copy link
Member

Hi @ianagne!

Thanks for this. Let's do a drop_table migration (in another PR).

@nickcharlton nickcharlton merged commit 7ba0f08 into thoughtbot:master May 3, 2017
jonatack pushed a commit to jonatack/administrate that referenced this pull request May 6, 2017
The delayed_job_active_record gem was removed in thoughtbot#845 and the jobs:work
rake task is no longer available.  Remove the worker setup from the
Procfile so rake jobs:work will not be called.
jonatack added a commit to jonatack/administrate that referenced this pull request May 6, 2017
* 'master' of https://github.com/thoughtbot/administrate:
  Tweak primary navigation (thoughtbot#861)
  Redesign focus outline styles (thoughtbot#863)
  Localise Dates (thoughtbot#570)
  Wrap unsupported form field notes in proper divs (thoughtbot#515)
  Update customizing_dashboards.md (thoughtbot#568)
  Wrap table cell contents instead of CSS truncation (thoughtbot#482)
  Remove worker from Procfile (thoughtbot#854)
  Remove dependency from ActionMailer (thoughtbot#851)
  Release version 0.6.0.
  Remove delayed_job_active_record dependency (thoughtbot#845)
  use the association includes in the has_many resources as well
  move include work to an association includes
  improve performance for n+1s
iarie pushed a commit to iarie/administrate that referenced this pull request Jun 17, 2017
The delayed_job_active_record gem was removed in thoughtbot#845 and the jobs:work
rake task is no longer available.  Remove the worker setup from the
Procfile so rake jobs:work will not be called.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants