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

Wrap 404 error so it can be detected later, instead of overwriting. #950

Merged
merged 1 commit into from
Nov 27, 2018

Conversation

nat-henderson
Copy link
Contributor

404 errors are overwritten so they can't be detected with handleNotFoundError - fix that, fix hashicorp/terraform-provider-google#2519.


[all]

[terraform]

Avoid overwriting errors so that they can be detected by handleNotFoundError.

[terraform-beta]

[ansible]

[inspec]

@modular-magician
Copy link
Collaborator

I am a robot that works on MagicModules PRs!

I built this PR into one or more PRs on other repositories, and when those are closed, this PR will also be merged and closed.
depends: hashicorp/terraform-provider-google-beta#156
depends: hashicorp/terraform-provider-google#2526

Copy link
Contributor

@danawillow danawillow left a comment

Choose a reason for hiding this comment

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

LGTM + downstreams.

@modular-magician modular-magician merged commit f6a2ac4 into master Nov 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Project log exclusion not recreated after being removed
4 participants