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 a invoke-after function for powershell plans #6685

Merged
merged 1 commit into from Jun 26, 2019

Conversation

@mwrock
Copy link
Contributor

commented Jun 25, 2019

fixes #6414

Signed-off-by: mwrock matt@mattwrock.com

@davidMcneil
Copy link
Contributor

left a comment

Looks good to me. Just a minor typo.

@@ -375,6 +375,17 @@ function Invoke-Before {
function Invoke-DefaultBefore {
}

# At this phase of the build, the package has been builtand installed,

This comment has been minimized.

Copy link
@davidMcneil

davidMcneil Jun 26, 2019

Contributor

A minor typo builtand to built and.

add a invoke-after function for powershell plans
Signed-off-by: mwrock <matt@mattwrock.com>

@mwrock mwrock force-pushed the invoke_after branch from c9fbd24 to f5e530c Jun 26, 2019

@mwrock mwrock merged commit 40f934c into master Jun 26, 2019

2 of 5 checks passed

buildkite/habitat-sh-habitat-master-verify Build #2522 started
Details
buildkite/habitat-sh-habitat-master-website Build #3044 started
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
DCO This commit has a DCO Signed-off-by
Details
expeditor/config-validation Validated your Expeditor config file
Details

@mwrock mwrock deleted the invoke_after branch Jun 26, 2019

chef-ci added a commit that referenced this pull request Jun 26, 2019

Update CHANGELOG.md with details from pull request #6685
Obvious fix; these changes are the result of automation not creative thinking.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.