Skip to content

Conversation

dillanb-hashi
Copy link
Collaborator

Description

Added dependencies for automation in the client setup script
Added an optional github token variable for running automation tests

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've documented the impact of any changes to security controls.
    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@dillanb-hashi dillanb-hashi requested a review from a team as a code owner October 10, 2025 16:20
@dillanb-hashi dillanb-hashi requested a review from a team October 10, 2025 16:21
@dillanb-hashi dillanb-hashi added the pr/no-milestone Ignores the Milestone Check label Oct 10, 2025
@dillanb-hashi dillanb-hashi requested a review from moduli October 10, 2025 19:40
@dillanb-hashi dillanb-hashi requested a review from moduli October 10, 2025 21:10
}

variable "github_token" {
description = "Github token"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could we also add the updated description here?

Comment on lines +45 to +47
if (${github_token} -ne "") {
git config --global url."https://oauth2:${github_token}@github.com".insteadOf "https://github.com"
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure if this is being executed. I tried it on my machine and I don't see it.

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

Labels

pr/no-milestone Ignores the Milestone Check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants