Skip to content

platform/4040: feat: add additional_transfer_job_excluded_prefixes va…#30

Merged
filippolmt merged 1 commit intomainfrom
feat/add_additional_transfer_job_excluded_prefixes
Jan 19, 2026
Merged

platform/4040: feat: add additional_transfer_job_excluded_prefixes va…#30
filippolmt merged 1 commit intomainfrom
feat/add_additional_transfer_job_excluded_prefixes

Conversation

@filippolmt
Copy link
Copy Markdown
Contributor

…riable for transfer job customization

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the additional_transfer_job_excluded_prefixes variable to allow users to exclude additional prefixes from the Google Cloud Storage transfer job without overriding the existing Drupal-specific defaults. The implementation maintains backward compatibility by providing an empty default and using concat() to merge the additional prefixes with the existing ones.

Key Changes:

  • Added additional_transfer_job_excluded_prefixes variable with an empty default list
  • Updated transfer job configuration to concatenate both prefix lists
  • Fixed trailing whitespace in main.tf and CHANGELOG.md

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
variables.tf Added new additional_transfer_job_excluded_prefixes variable with type list(string) and empty default
main.tf Updated exclude_prefixes to concat both the original and additional prefix lists; fixed trailing whitespace in comment
README.md Added documentation for the new variable in the inputs table (alphabetically sorted)
CHANGELOG.md Added release notes for version 0.11.0 documenting the new variable; fixed trailing whitespace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@andypanix andypanix left a comment

Choose a reason for hiding this comment

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

LGTM

@filippolmt filippolmt merged commit 4267aba into main Jan 19, 2026
7 checks passed
@filippolmt filippolmt deleted the feat/add_additional_transfer_job_excluded_prefixes branch January 19, 2026 07:41
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.

3 participants