Skip to content

fix: standardize variable names in postBuild substitution#2123

Merged
avnes merged 1 commit intomasterfrom
hotfix/standardize-vars-postbuild-subst
Dec 2, 2025
Merged

fix: standardize variable names in postBuild substitution#2123
avnes merged 1 commit intomasterfrom
hotfix/standardize-vars-postbuild-subst

Conversation

@avnes
Copy link
Copy Markdown
Contributor

@avnes avnes commented Dec 2, 2025

Describe your changes

This pull request makes a minor update to the variable names used in the Helm chart dependencies specification for 1Password Connect. The change improves naming consistency by updating the variable prefixes to match the rest of the codebase.

  • Updated variable names from fluxcd_aws_region and fluxcd_aws_workload_account_id to flux_aws_region and flux_workload_account_id in _sub/security/helm-1password-connect/dependencies.tf for consistency.

Checklist before requesting a review

Is it a new release?

  • Apply a release tag release:(major|minor|patch), following semantic versioning in this guide or norelease if there is no changes to the Terraform code

@avnes avnes requested a review from a team as a code owner December 2, 2025 15:30
@avnes avnes added the release:patch Triggers a patch release label Dec 2, 2025
@avnes avnes merged commit d75f0d3 into master Dec 2, 2025
16 checks passed
@avnes avnes deleted the hotfix/standardize-vars-postbuild-subst branch December 2, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:patch Triggers a patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants