Skip to content

enables multiple includes #707

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

enables multiple includes #707

wants to merge 2 commits into from

Conversation

twang817
Copy link

Allows for multiple includes. Should fix #303.

path_relative_from_include() and path_relative_to_include include tested. Behavior should be to evaluate within the context of the included file.

@twang817 twang817 force-pushed the multiple_includes branch from 9cb7188 to ccede80 Compare May 17, 2019 14:19
@brikis98
Copy link
Member

Thanks for the PR! I'm a bit buried now and will review this later on to think through the implications.

@brikis98 brikis98 mentioned this pull request Jun 5, 2019
Open
@apottere
Copy link
Contributor

How does this work with *-all commands? Is there a way to either mark files as intermediary or mark them as leaves?

@brikis98
Copy link
Member

@twang817 When you get a chance, please rebased this PR on master. Note that in v0.19.0, Terragrunt config parsing had a major refactor to support Terraform 0.12 and move to HCL2.

@alberts-s alberts-s mentioned this pull request Jun 26, 2019
Open
@icereed
Copy link

icereed commented Mar 18, 2020

Would be happy about this feature!

@asafha-etorox
Copy link

Until we get the imports block, I would be majorly super-duper happy to see it revisited!

@maulik-vimond
Copy link

It would be great to get this one merged. We want the child modules to include some extra files (that generate TF config) and that is not possible at present.

@erdnaxeli
Copy link

erdnaxeli commented Jul 18, 2023 •
edited
Loading

Hi, what is blocking to see this merged?

@notwaf
Copy link

notwaf commented Feb 8, 2025

can we get this merged in? been over a year :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@autero1 autero1 Awaiting requested review from autero1

@brikis98 brikis98 Awaiting requested review from brikis98

@eak12913 eak12913 Awaiting requested review from eak12913

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Only one level of includes is allowed
8 participants
@twang817 @brikis98 @apottere @icereed @asafha-etorox @maulik-vimond @erdnaxeli @notwaf