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

Changed wording in docs for Task.retry() to mention the fact that it requeues #4933

Merged
merged 2 commits into from Jul 28, 2018

Conversation

Projects
None yet
2 participants
@dashdanw
Contributor

dashdanw commented Jul 27, 2018

I found it hard to find this information on the documentation and it was confirmed to me by users of the freenode IRC channel. I thought it might be helpful to change the wording slightly.

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov Jul 27, 2018

Codecov Report

Merging #4933 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4933   +/-   ##
=======================================
  Coverage   82.89%   82.89%           
=======================================
  Files         140      140           
  Lines       15958    15958           
  Branches     1998     1998           
=======================================
  Hits        13229    13229           
  Misses       2533     2533           
  Partials      196      196
Impacted Files Coverage Δ
celery/app/task.py 96.51% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2e2ffe...785d6a1. Read the comment docs.

codecov commented Jul 27, 2018

Codecov Report

Merging #4933 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4933   +/-   ##
=======================================
  Coverage   82.89%   82.89%           
=======================================
  Files         140      140           
  Lines       15958    15958           
  Branches     1998     1998           
=======================================
  Hits        13229    13229           
  Misses       2533     2533           
  Partials      196      196
Impacted Files Coverage Δ
celery/app/task.py 96.51% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2e2ffe...785d6a1. Read the comment docs.

@auvipy

auvipy approved these changes Jul 28, 2018

@auvipy auvipy merged commit bff690d into celery:master Jul 28, 2018

4 checks passed

codecov/patch Coverage not affected when comparing e2e2ffe...785d6a1
Details
codecov/project 82.89% remains the same compared to e2e2ffe
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

tlwr added a commit to tlwr/celery that referenced this pull request Aug 15, 2018

Changed wording in docs for Task.retry() to mention the fact that it …
…requeues (#4933)

* changed wording for autodocs

* Updated wording to be more descriptive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment