Update the module API
skx opened this issue · comments
Steve Kemp commented
It is odd that the Check
method doesn't have a pointer to the environment, but that the Execute
one does.
Although it is more boiler-plate I think the correct solution here is to pass the environment to the module constructor - and remove it as an argument to the Execute.
Once this is done it'll be time for a new release.