Skip to content

Conversation

timofurrer
Copy link
Member

@timofurrer timofurrer commented Oct 14, 2022

For some reason the check if the file exists was matching some text in the error message - however it should check the status code, like everywhere else.

Closes #1260

@timofurrer timofurrer added this to the v3.19.0 milestone Oct 14, 2022
@timofurrer timofurrer self-assigned this Oct 14, 2022
@timofurrer timofurrer linked an issue Oct 14, 2022 that may be closed by this pull request
@github-actions github-actions bot added provider resource Adds or modifies a resource size/XS labels Oct 14, 2022
Copy link
Collaborator

@RicePatrick RicePatrick left a comment

Choose a reason for hiding this comment

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

Hey @timofurrer - Added the "closes" link to your initial description, otherwise LGTM. I'll approve here, and if you're good with closing it, go ahead and merge. Just in case you didn't want to close the issue, I'll leave this open for now :)

@timofurrer timofurrer merged commit e189ddb into main Oct 24, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

provider resource Adds or modifies a resource size/XS

Development

Successfully merging this pull request may close these issues.

Deleting a managed file from the repo leads to Terraform error

2 participants