Skip to content

Commit

Permalink
fix: Removed tf lints warning
Browse files Browse the repository at this point in the history
  • Loading branch information
13archit committed Sep 12, 2024
1 parent 64702d4 commit 00a8d9f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 14 deletions.
4 changes: 2 additions & 2 deletions examples/main.tf
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
provider "azurerm" {
features {}
subscription_id = "068245d4-3c94-42fe-9c4d-9e5e1cabc60c"
subscription_id = "000000-11111-1223-XXX-XXXXXXXXXXXX"
}

locals {
Expand All @@ -14,7 +14,7 @@ module "resource_group" {
version = "1.0.2"
name = local.name
environment = local.environment
location = "Canada Central"
location = local.location
}

##-----------------------------------------------------------------------------
Expand Down
12 changes: 0 additions & 12 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,6 @@ variable "enabled" {
description = "Set to false to prevent the module from creating any resources."
}

variable "tags" {
type = map(string)
default = {}
description = "A map of tags to add to all resources"
}

variable "sentinel_enabled" {
type = bool
default = true
Expand All @@ -54,12 +48,6 @@ variable "log_analytics_workspace_id" {
description = "The ID of the Log Analytics Workspace this Sentinel MS Security Incident Alert Rule belongs to. Changing this forces a new Sentinel MS Security Incident Alert Rule to be created."
}

variable "sentinel_workspace_name" {
type = string
default = ""
description = "The name which should be used for this Sentinel MS Security Incident Alert Rule. Changing this forces a new Sentinel MS Security Incident Alert Rule to be created."
}

variable "product_filter" {
type = list(string)
default = ["Microsoft Cloud App Security"]
Expand Down

0 comments on commit 00a8d9f

Please sign in to comment.