Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add type descriptions for complex input #65

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

automaticgiant
Copy link

Description

attempt describing datasources input with complex type

it's my first time using these, and i'm also not 100% sure it captures everything about the module. just hoping to move the bar a little bit.

Motivation and Context

unclear what options are supported for data sources; caused #64 to be made then closed. documentation followup.

Breaking Changes

none

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
    • at least passed validation, but configuration is not applyable in my account as is - tweaked zone but gave up after acm validation error
  • I have executed pre-commit run -a on my pull request
hunter.morgan@INV-04069 terraform-aws-appsync % docker run -e "USERID=$(id -u):$(id -g)" -v $(pwd):/lint -w /lint ghcr.io/antonbabenko/pre-commit-terraform:$TAG run -a

[INFO] Initializing environment for https://github.com/antonbabenko/pre-commit-terraform.
[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
Terraform fmt............................................................Passed
Terraform wrapper with for_each in module................................Passed
Terraform docs...........................................................Passed
Terraform validate with tflint...........................................Passed
Terraform validate.......................................................Failed
- hook id: terraform_validate
- exit code: 1

Validation failed: examples/complete
╷
│ Error: missing or corrupted provider plugins:
│   - registry.terraform.io/hashicorp/aws: there is no package for registry.terraform.io/hashicorp/aws 5.65.0 cached in .terraform/providers
│   - registry.terraform.io/hashicorp/random: there is no package for registry.terraform.io/hashicorp/random 3.6.2 cached in .terraform/providers
│ 
│ 
╵

check for merge conflicts................................................Passed
fix end of files.........................................................Passed
trim trailing whitespace.................................................Passed
hunter.morgan@INV-04069 terraform-aws-appsync % terraform validate
Success! The configuration is valid.

hunter.morgan@INV-04069 terraform-aws-appsync % 

@automaticgiant automaticgiant mentioned this pull request Aug 30, 2024
4 tasks
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