-
-
Notifications
You must be signed in to change notification settings - Fork 245
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
Remove unused dependencies #156
Labels
bug
🐛 An issue with the system
Comments
Merged
bion
referenced
this issue
in civiform/cloud-deploy-infra
Apr 3, 2023
…126) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [cloudposse/ecs-container-definition/aws](https://registry.terraform.io/modules/cloudposse/ecs-container-definition/aws) ([source](https://togithub.com/cloudposse/terraform-aws-ecs-container-definition)) | module | patch | `0.58.1` -> `0.58.2` | --- ### Release Notes <details> <summary>cloudposse/terraform-aws-ecs-container-definition</summary> ### [`v0.58.2`](https://togithub.com/cloudposse/terraform-aws-ecs-container-definition/releases/tag/0.58.2) [Compare Source](https://togithub.com/cloudposse/terraform-aws-ecs-container-definition/compare/0.58.1...0.58.2) <details> <summary>Remove unused provider @​pguinard-public-com (#​157)</summary> #### what * Remove unused provider #### why * Do not download the [provider for local.file and local.sensitive_file](https://registry.terraform.io/providers/hashicorp/local/latest/docs/resources/file) as this is not required for the module to run * This makes terraform init run faster as well as download resources that are not used #### references * Local provider that is being defined ([https://github.com/cloudposse/terraform-aws-ecs-container-definition/issues/156](https://togithub.com/cloudposse/terraform-aws-ecs-container-definition/issues/156)) * closes #​156 </details> <details> <summary>git.io->cloudposse.tools update @​dylanbannon (#​152)</summary> #### what and why Change all references to `git.io/build-harness` into `cloudposse.tools/build-harness`, since `git.io` redirects will stop working on April 29th, 2022. #### References - DEV-143 </details> #### 🤖 Automatic Updates <details> <summary>Update README.md and docs @​cloudpossebot (#​155)</summary> #### what This is an auto-generated PR that updates the README.md and docs #### why To have most recent changes of README.md and doc from origin templates </details> </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/civiform/cloud-deploy-infra). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC4xNTIuNCIsInVwZGF0ZWRJblZlciI6IjM0LjE1Mi40In0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the Bug
required_providers { local ... } is not required as locals are built into terraform. The local provider provides the following:
Expected Behavior
Not install a provider that is not required in order to speed up the terraform init process.
Steps to Reproduce
Sample terraform code
Screenshots
If applicable, add screenshots or logs to help explain your problem.
Environment (please complete the following information):
Anything that will help us triage the bug will help. Here are some ideas:
Additional Context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: