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 archive_dir property to be used with source_dir that will prefix … #125

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jhancock93
Copy link

…all content

This can be used to address issue issue #96 and feature request #37. To keep the parent directory, it just must be specified in archive_dir property. archive_dir can have multiple parent levels, if needed. As this uses a new field, it does not regress any current behavior.

@hashicorp-cla
Copy link

hashicorp-cla commented Jan 14, 2022

CLA assistant check
All committers have signed the CLA.

@jhancock93
Copy link
Author

Now that I've issued this, I see this is similar to #113. If that can address the other issues, then this PR is not needed.

@jhancock93
Copy link
Author

@teamterraform, not sure what the intake process is here. Is a PR reviewer typically assigned automatically? I do see a related PR but it seems to be very stale.

@jhancock93 jhancock93 mentioned this pull request Mar 6, 2022
@jhancock93
Copy link
Author

@teamterraform, anything we can do to move this forward?

@@ -78,6 +78,8 @@ NOTE: One of `source`, `source_content_filename` (with `source_content`), `sourc

* `source_dir` - (Optional) Package entire contents of this directory into the archive.

Choose a reason for hiding this comment

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

Missing period.

Choose a reason for hiding this comment

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

Meant to link to

* `archive_dir` - (Optional) When used with source_dir, the top-level directories inside the archive that will contain all the contents

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants