-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Replace multierror #1703
Replace multierror #1703
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution! Changes LGTM! Let me kick off a regression build, and if it passes, we can merge this in.
Looks like there is a build failure. Specifically, the check at https://github.com/gruntwork-io/terragrunt/blob/pull-request-1703/test/integration_test.go#L1502 and https://github.com/gruntwork-io/terragrunt/blob/pull-request-1703/test/integration_test.go#L1711 are failing with:
|
@yorinasub17 my bad, I thought it was a glitch during my local run :) Fixed |
Thanks for updating! Just kicked off a build. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build passed, so will go ahead and merge this in. Thanks again for your contribution!
This PR addresses #1482
Here is the changes in output.
Previous output:
New output:
or more detailed example
Previous output:
New output: