This is a nit, but we should be consistent. The watcher.applier.actions.base code has:
@abc.abstractmethod
def precondition(self):
"""Hook: called before the execution of an action
This method can be used to perform some initializations or to make
some more advanced validation on its input parameters. So if you wish
to block its execution based on this factor, `raise` the related
exception.
"""
raise NotImplementedError()
@abc.abstractmethod
def postcondition(self):
"""Hook: called after the execution of an action
This function is called regardless of whether an action succeded or
not. So you can use it to perform cleanup operations.
"""
raise NotImplementedError()
And in other areas of similar patterns, we use pre_condition and post_condition. We should clean this up and use consistent formatting of the method names (read: rename these to use the '_' delimiter).
Fix proposed to branch: master
Review: https:/