Skip to content

Remove pre_process and post_process from the ExecutionHooksHint class (move content to JobWrapper) #72

@aldbr

Description

@aldbr

Now that ExecutionHooksHint child classes embed a list of Pre and PostExecution commands, I think we should move the pre_process and post_process methods to the JobWrapper.

It would better separate responsibilities:

  • the ExecutionHooksHint child classes would focus on declaring the commands they are going to run
  • the JobWrapper would handle executing them

Metadata

Metadata

Assignees

No one assigned

    Labels

    points:1Trivial, 1 day, very clear

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions