-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Prepare for upcoming Sidekiq::Config redesign #5340
Conversation
Adjust the server internals to use a config object rather than refering directly to the Sidekiq module.
For 6.x, Sidekiq::Components will be passed the Sidekiq module itself as a parameter. This will become a |
Let’s discuss deeper design questions Monday.
…On Sun, May 29, 2022 at 16:29 Sam ***@***.***> wrote:
Whats your motivation behind using a config object over a sidekiq module?
—
Reply to this email directly, view it on GitHub
<#5340 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAAWX4UPSJOKWY3CIOMDEDVMP4VPANCNFSM5WW3ZF2Q>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sorry, forgot tomorrow is a holiday. Tuesday! |
No problem! See ya tomorrow!
…On Sun, May 29, 2022 at 21:27 Mike Perham ***@***.***> wrote:
Sorry, forgot tomorrow is a holiday. Tuesday!
—
Reply to this email directly, view it on GitHub
<#5340 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFL7TVQ2SAPQIZO5VVDXNLVMQ7SBANCNFSM5WW3ZF2Q>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
I'm interested in contributing to this implementation as well. |
@TonsOfFun I don't have a good process for collaborating on this. |
The removal of What is the recommended way to provide extra context to exceptions in Sidekiq 6.x? In Discourse we've been using this: |
@CvX Please do not litter closed issues with your own problems. Open your own issue. I would be happy to help. |
Sidekiq starts to store/pass configurations differently, which doesn't affect the SDK itself. But we need to update the test setup accordingly. Related PR:sidekiq/sidekiq#5340
Sidekiq starts to store/pass configurations differently, which doesn't affect the SDK itself. But we need to update the test setup accordingly. Related PR:sidekiq/sidekiq#5340
Sidekiq starts to store/pass configurations differently, which doesn't affect the SDK itself. But we need to update the test setup accordingly. Related PR:sidekiq/sidekiq#5340
Sidekiq starts to store/pass configurations differently, which doesn't affect the SDK itself. But we need to update the test setup accordingly. Related PR:sidekiq/sidekiq#5340
Removed util support sidekiq/sidekiq#5340
Adjust the server internals to use a config object rather than refering directly to the Sidekiq module.