Skip to content

docs: add a section on getting the client working with celery #210

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
May 31, 2022
Merged

docs: add a section on getting the client working with celery #210

merged 3 commits into from
May 31, 2022

Conversation

sighphyre
Copy link
Member

Adds documentation on how to get this project working with Celery

@coveralls
Copy link

coveralls commented May 31, 2022 •
edited
Loading

Coverage Status

Coverage remained the same at 96.089% when pulling 860ca40 on docs/celery into 33cb928 on main.

@sighphyre
Copy link
Member Author

Lots of users are still struggling to get Celery working properly with the Python SDK

thomasheartman
thomasheartman requested changes May 31, 2022
View reviewed changes
Copy link
Contributor

@thomasheartman thomasheartman left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good one! 🙌🏼 Should be great to get this in 😄 I've added a small rewording suggestion and have a little discussion point before we do, though.

thomasheartman
thomasheartman approved these changes May 31, 2022
View reviewed changes
Copy link
Contributor

@thomasheartman thomasheartman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! 🚂

@sighphyre sighphyre merged commit 736b13f into main May 31, 2022
@sighphyre sighphyre deleted the docs/celery branch May 31, 2022 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thomasheartman thomasheartman thomasheartman approved these changes

@ivanklee86 ivanklee86 Awaiting requested review from ivanklee86

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sighphyre @coveralls @thomasheartman