Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add notes to doc about prerRun and postRun function execution condition #2040

Closed
wants to merge 3 commits into from

Conversation

haoming29
Copy link
Contributor

According to #2039, As per code, if the current command does not have a Run function defined, the PreRun or PostRun function will not be executed at current command level, including PersistentPre

@CLAassistant
Copy link

CLAassistant commented Oct 7, 2023

CLA assistant check
All committers have signed the CLA.

@haoming29 haoming29 closed this Oct 7, 2023
@haoming29 haoming29 deleted the add-notes-to-preRun-functions branch October 7, 2023 04:36
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants