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

Support loading options from config/delayed_job.yml #1047

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

excid3
Copy link

@excid3 excid3 commented Aug 25, 2018

One of the nice things about Sidekiq is being able to save all your commandline options in a config file so you can reuse that everywhere. I just wanted to add the same functionality to delayed_job so that it would be easier to manage and track your options in git as well.

@caseyprovost
Copy link

👍 Love it!

@peterschols
Copy link

This is fantastic, thanks a lot!
I hope the maintainers merge this into DJ soon.
Thanks!

@betamatt
Copy link
Collaborator

I'm personally against checked-in config files but lots of people seem to like them. I don't have strong feelings about whether this is in scope or not for DJ.

@peterschols
Copy link

@betamatt are there any updates on the status of this PR?

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.

None yet

4 participants