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

Add Sidekiq::Job #4956

Closed
wants to merge 3 commits into from
Closed

Add Sidekiq::Job #4956

wants to merge 3 commits into from

Conversation

mperham
Copy link
Collaborator

@mperham mperham commented Aug 7, 2021

  • Provide Sidekiq::Worker alias for backwards compat
  • Rename API's Sidekiq::Job to Sidekiq::JobRecord
  • Adjust internals to use Sidekiq::JobRecord

Implements #4955

* Provide Sidekiq::Worker alias for backwards compat
* Rename API's Sidekiq::Job to Sidekiq::JobRecord
* Adjust internals to use Sidekiq::JobRecord
@mperham mperham changed the title Add Sidekiq::Job, see #4955 Add Sidekiq::Job Aug 7, 2021
Minor breakage due to removing constant DEFAULT_WORKER_OPTIONS. Users
should use Sidekiq.default_job_options = { ... } instead.
Changes.md Show resolved Hide resolved
@mperham
Copy link
Collaborator Author

mperham commented Aug 20, 2021

We'll do this slower, in multiple stages.

@mperham mperham closed this Aug 20, 2021
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

2 participants