Skip to content

Comments

feat(command): add support for post-run hooks#12

Merged
arikkfir merged 1 commit intomainfrom
arikkfir/separate-action-and-run-hooks
Jun 14, 2024
Merged

feat(command): add support for post-run hooks#12
arikkfir merged 1 commit intomainfrom
arikkfir/separate-action-and-run-hooks

Conversation

@arikkfir
Copy link
Owner

@arikkfir arikkfir commented Jun 14, 2024

This change adds support for providing post-run command hooks, enabling commands to be (when needed) initialization & tear-down wrappers around concrete sub-commands.

Unlike pre-run hooks (which are run in order, depth-last, before actual command execution), post-run hooks are run depth-first (reverse order).

Both the pre & post run hooks can be structs that have their own flags
and configurations.

@arikkfir arikkfir added the feature New features and functionality label Jun 14, 2024
@arikkfir arikkfir self-assigned this Jun 14, 2024
@arikkfir arikkfir force-pushed the arikkfir/separate-action-and-run-hooks branch from 7264c74 to b5e5560 Compare June 14, 2024 11:49
@arikkfir arikkfir changed the title feat(command): add support for hooks-only intermediate commands without logic feat(command): add support for post-run hooks Jun 14, 2024
This change adds support for providing post-run command hooks, enabling
commands to be (when needed) initialization & tear-down wrappers around
concrete sub-commands.

Unlike pre-run hooks (which are run in order, depth-last, before actual
command execution), post-run hooks are run depth-first (reverse order).

Both the pre & post run hooks can be structs that have their own flags
and configurations.
@arikkfir arikkfir force-pushed the arikkfir/separate-action-and-run-hooks branch from b5e5560 to 3e47d12 Compare June 14, 2024 11:55
@arikkfir arikkfir enabled auto-merge (rebase) June 14, 2024 11:56
@arikkfir arikkfir merged commit 53de84a into main Jun 14, 2024
@arikkfir arikkfir deleted the arikkfir/separate-action-and-run-hooks branch June 14, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New features and functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant