Skip to content

Strip newline in Terraform command options#254

Merged
ieuanb74 merged 2 commits intomasterfrom
ieuanb74-patch-1
Mar 2, 2026
Merged

Strip newline in Terraform command options#254
ieuanb74 merged 2 commits intomasterfrom
ieuanb74-patch-1

Conversation

@ieuanb74
Copy link
Copy Markdown
Contributor

@ieuanb74 ieuanb74 commented Mar 2, 2026

Jira link

Change description

  • Newline is being inserted in the commandOptions parameter for Terraform plan/apply
  • Stripping to resolve issue
  • Also fixing retry indentation

Testing done

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@ieuanb74 ieuanb74 marked this pull request as ready for review March 2, 2026 09:51
@ieuanb74 ieuanb74 requested a review from a team as a code owner March 2, 2026 09:51
@ieuanb74 ieuanb74 merged commit ad9b6d3 into master Mar 2, 2026
1 check passed
@ieuanb74 ieuanb74 deleted the ieuanb74-patch-1 branch March 2, 2026 09:56
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.

2 participants