Skip to content

Conversation

@esiebes
Copy link

@esiebes esiebes commented Dec 4, 2017

#61
resolution for issue 61. In our company me an my colleagues also noticed that cchoco always returned succes even when chocolatey gave an error. This fix wil throw an error if chocolatey gives an error.

@esiebes esiebes changed the title add invoke-chocolatey which throws error on choco error Fix issue #61 add invoke-chocolatey which throws error on choco error Dec 8, 2017
}
}
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably can't influence the exit code of the run can it?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exit code is now handled in pull request 103

(
# Param1 help description
[Parameter(Position=0)]
$pParams
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we just call this arguments?

function Invoke-Chocolatey
{
[CmdletBinding()]
[Alias("IChoco")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's not add an alias

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alias removed in pullrequest 103

{
[CmdletBinding()]
[Alias("IChoco")]
[OutputType([int])]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How will this work with choco list -lo -r that pipes the text output?

@esiebes
Copy link
Author

esiebes commented Dec 13, 2017

i

@esiebes esiebes closed this Dec 13, 2017
@esiebes
Copy link
Author

esiebes commented Dec 13, 2017

please see pullrequest 103.

@ferventcoder
Copy link
Member

Closed in favor of #103

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.

3 participants