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 delayed_job_active_record dependency #845

Merged

Conversation

ianagne
Copy link
Contributor

@ianagne ianagne commented Apr 29, 2017

The delayed_job_active_record gem was added by suspenders automatically
during the generation of this gem. The delayed_job_active_record gem is
not currently used.

Of note, delayed_job_active_record does not currently support Rails 5.1. Removing the dependency has the side bonus of moving us closer to supporting Rails 5.1.

  • Remove delayed_job_active_record dependency
  • Re-generate appraisal gemfiles to exclude delayed_job_active_record
  • Remove delayed_job binstubs
  • Remove BackgroundJobs spec support module

The delayed_job_active_record gem was added by suspenders automatically
during the generation of this gem.  The delayed_job_active_record is
not currently used.

* Remove delayed_job_active_record dependency
* Re-generate appraisal gemfiles to exclude delayed_job_active_record
* Remove delayed_job binstubs
* Remove BackgroundJobs spec support module
@pustomytnyk
Copy link
Contributor

"Some checks were not successful"
I've created issue #846 about that test being failed sometimes.

@BenMorganIO
Copy link
Collaborator

Yessssss one less gem!!!

@BenMorganIO BenMorganIO reopened this Apr 29, 2017
@nickcharlton
Copy link
Member

Thanks!

The tests are failing because of a known flakey test, so I'm going to go ahead and merge anyway.

@nickcharlton nickcharlton merged commit e5e9285 into thoughtbot:master May 2, 2017
ianagne added a commit to ianagne/administrate that referenced this pull request May 3, 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.
nickcharlton pushed a commit that referenced this pull request 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.
jonatack pushed a commit to jonatack/administrate that referenced this pull request May 6, 2017
The delayed_job_active_record gem was added by suspenders automatically
during the generation of this gem.  The delayed_job_active_record is
not currently used.

* Remove delayed_job_active_record dependency
* Re-generate appraisal gemfiles to exclude delayed_job_active_record
* Remove delayed_job binstubs
* Remove BackgroundJobs spec support module
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
@deivid-rodriguez deivid-rodriguez mentioned this pull request May 8, 2017
iarie pushed a commit to iarie/administrate that referenced this pull request Jun 17, 2017
The delayed_job_active_record gem was added by suspenders automatically
during the generation of this gem.  The delayed_job_active_record is
not currently used.

* Remove delayed_job_active_record dependency
* Re-generate appraisal gemfiles to exclude delayed_job_active_record
* Remove delayed_job binstubs
* Remove BackgroundJobs spec support module
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.

4 participants