Skip to content

fix ResourceGroupExists#1552

Merged
james00012 merged 2 commits intomainfrom
fix-resource-group
May 9, 2025
Merged

fix ResourceGroupExists#1552
james00012 merged 2 commits intomainfrom
fix-resource-group

Conversation

@james00012
Copy link
Copy Markdown
Contributor

Description

Fixes #1537.

TODOs

Read the Gruntwork contribution guidelines.

  • Update the docs.
  • Run the relevant tests successfully, including pre-commit checks.
  • Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
  • Include release notes. If this PR is backward incompatible, include a migration guide.
  • Make a plan for release of the functionality in this PR. If it delivers value to an end user, you are responsible for ensuring it is released promptly, and correctly. If you are not a maintainer, you are responsible for finding a maintainer to do this for you.

Release Notes (draft)

Added / Removed / Updated [X].

Migration Guide

@james00012 james00012 self-assigned this May 7, 2025
yhakbar
yhakbar previously approved these changes May 7, 2025
denis256
denis256 previously approved these changes May 7, 2025
@james00012 james00012 dismissed stale reviews from denis256 and yhakbar via d683c05 May 8, 2025 18:24
@james00012 james00012 force-pushed the fix-resource-group branch from aee3bb3 to d683c05 Compare May 8, 2025 18:24
@james00012
Copy link
Copy Markdown
Contributor Author

Hi @yhakbar or @denis256, require another approval after merging some conflict.

@james00012 james00012 merged commit 81ff891 into main May 9, 2025
2 of 3 checks passed
@denis256 denis256 deleted the fix-resource-group branch May 12, 2025 16:44
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.

azure.ResourceGroupExistsE returns an error if the resource group doesn't exist, rather than false

3 participants