Skip to content

Conversation

mcalhoun
Copy link
Contributor

@mcalhoun mcalhoun commented Feb 8, 2024

what

Initial module implementation

value = each.value.value
}

data "aws_ssm_parameter" "read_kv_pair" {
Copy link

Choose a reason for hiding this comment

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

⚠️ [tflint] reported by reviewdog 🐶
Missing version constraint for provider "aws" in required_providers

main.tf Outdated
locals {
example = format("%v %v", var.example, join("", random_integer.example[*].result))
// Build a map of keys to read from the key/value store and the key they should be output to, with a shape of: { output_key_name = key_path_to_read_from }
vals_to_read = {
Copy link

Choose a reason for hiding this comment

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

⚠️ [tflint] reported by reviewdog 🐶
local.vals_to_read is declared but not used

@@ -1,14 +1,3 @@
variable "region" {
Copy link

Choose a reason for hiding this comment

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

⚠️ [tflint] reported by reviewdog 🐶
variable "region" is declared but not used

@@ -1,3 +1,3 @@
provider "local" {
# Configuration options
provider "aws" {
Copy link

Choose a reason for hiding this comment

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

⚠️ [tflint] reported by reviewdog 🐶
Missing version constraint for provider "aws" in required_providers

@mcalhoun mcalhoun requested review from a team as code owners February 8, 2024 16:41
@mcalhoun mcalhoun requested review from joe-niland and kevcube and removed request for a team February 8, 2024 16:41
nullable = false
}

variable "ssm_enabled" {
Copy link

Choose a reason for hiding this comment

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

⚠️ [tflint] reported by reviewdog 🐶
variable "ssm_enabled" is declared but not used

Copy link

mergify bot commented Mar 25, 2024

This pull request is now in conflict. Could you fix it @mcalhoun? 🙏

1 similar comment
Copy link

mergify bot commented Jul 18, 2024

This pull request is now in conflict. Could you fix it @mcalhoun? 🙏

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.

1 participant