search

mozilla / active-data-recipes

A repository of various activedata queries and recipes
Mozilla Public License 2.0
8 stars 24 forks source link

Modify run_query function (use args instead of **context) #95

Closed TrangNguyenBC closed 5 years ago

TrangNguyenBC commented 5 years ago

Hi @ahal The extract_arguments(func, call, members=None) will read run function and extract run contexts from attributes of first arguments (in this case, it is args). Should we change the order of the arguments run_query instead?

ahal commented 5 years ago

Sure, the more important piece is that the order is consistent. Though couldn't extract_arguments just look at the second variable instead?

TrangNguyenBC commented 5 years ago

Yes, I updated :-)

ahal commented 5 years ago

Thanks!