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.

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

Adjust behaviour of task status reporting #258

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Adjust behaviour of task status reporting #258

wants to merge 5 commits into from

Commits on Aug 24, 2018

  1. Adjust task status reporting to take kwargs 

    The current implementation here is a bit clunky, and I see no particular reason why we can't just take keyword arguments. That means that using it is as simple as `self.report_status(progress=0.5, message='Glass half full')`, instead of the current `self.report_status(dict(progress=0.5, message='Glass half empty'))`.
    @bsmithyman
    bsmithyman authored Aug 24, 2018
    Configuration menu
    Copy the full SHA
    33009ac View commit details
    Browse the repository at this point in the history

  2. Update test_task.py

    @bsmithyman
    bsmithyman authored Aug 24, 2018
    Configuration menu
    Copy the full SHA
    6cbfb90 View commit details
    Browse the repository at this point in the history

  3. Fix horrid typo

    @bsmithyman
    bsmithyman authored Aug 24, 2018
    Configuration menu
    Copy the full SHA
    fb9e321 View commit details
    Browse the repository at this point in the history

  4. Fix reciprocal horrid typo

    @bsmithyman
    bsmithyman authored Aug 24, 2018
    Configuration menu
    Copy the full SHA
    d2e7ac1 View commit details
    Browse the repository at this point in the history

  5. Fix test for error type

    @bsmithyman
    bsmithyman authored Aug 24, 2018
    Configuration menu
    Copy the full SHA
    09fe1bb View commit details
    Browse the repository at this point in the history