This repository was archived by the owner on Jan 31, 2019. It is now read-only.

Update invoke syntax #1

Merged

sloria merged 2 commits into sloria:master from anhdat:update_invoke_syntax

Jan 8, 2017

Contributor

anhdat commented Jan 8, 2017

Hi cloria, since 0.13.0, invoke requires all task must be contextualized, see more. This PR tends to fix that.

[Feature] #114: Ripped off the band-aid and removed non-contextualized tasks as an option; all tasks must now be contextualized (defined as def mytask(context, ...) - see Defining and running task functions) even if not using the context. This simplifies the implementation as well as users’ conceptual models. Thanks to Bay Grabowski for the patch.

Please let me know if you need any modification. Thanks.

anhdat added 2 commits

January 8, 2017 15:33


          Remove unused sys

a7a08da


          Update invoke syntax with context

62a9698

Owner

sloria commented Jan 8, 2017

Thanks! I'll go ahead and pin the invoke version too.

sloria merged commit 7b60d6a into sloria:master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet