Skip to content

feat: azure rg building block#175

Open
younGihan wants to merge 1 commit into
mainfrom
feature/azure-resource-group
Open

feat: azure rg building block#175
younGihan wants to merge 1 commit into
mainfrom
feature/azure-resource-group

Conversation

@younGihan
Copy link
Copy Markdown
Contributor

No description provided.

@younGihan younGihan requested a review from florianow May 19, 2026 14:22
@aws-amplify-eu-central-1
Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-175.d1o16zfeoh2slu.amplifyapp.com

Copy link
Copy Markdown
Collaborator

@florianow florianow left a comment

Choose a reason for hiding this comment

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

one comment

@@ -0,0 +1,4 @@
provider "azurerm" {
subscription_id = var.subscription_id
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

use the ENV var pls and not adding vars in the provider. we dont need them there

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

you can also remove it from the the vars. We can use the tenant id.

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.

2 participants