fix(terraform): Ignored `--external-modules-download-path` flag
User description
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Description
Fixes a bug where --external-modules-download-path is ignored.
Fixes #6675
Checklist:
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [x] I have made corresponding changes to the documentation
- [x] I have added tests that prove my feature, policy, or fix is effective and works
- [x] New and existing tests pass locally with my changes
Generated description
Below is a concise technical summary of the changes proposed in this PR:
Implements the --external-modules-download-path flag functionality in the Terraform plan runner. Modifies the RunnerRegistry and TFParser classes to accept and use the external modules download path parameter. Adds a test case to verify the correct usage of the new parameter.
Latest Contributors(2)
| User | Commit | Date |
|---|---|---|
| lirshindalman | fix-terraform_plan-run... | December 01, 2024 |
| [email protected] | feat-arm-unsupported-m... | October 15, 2024 |
Hey @apgrucza, Thanks for the contribution! Could you please merge the latest changes from the main branch into your branch? Thanks!
Sorry for the delay @MaryArmaly, I've now rebased with the latest changes from main
@apgrucza Can we look at reviving this pull request? I would really like to see this error get fixed
Still waiting for it to be merged @Guitrum. I've rebased again to resolve conflicts. @MaryArmaly can we get this merged now?
I've updated the branch & run tests.
Hey @apgrucza, Since all the tests have passed and there are no conflicts with the main branch, I've merged this PR. Thanks for contributing!