Skip to content

Conversation

@Kevin-DeJong
Copy link
Collaborator

No description provided.


for tName, tCase := range testCases {
t.Run(tName, func(t *testing.T) {
resourceType := "org_setting"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Move up out of all for loops, please update the other resources in this pr as well.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Okay, done for realsies now.

Copy link
Collaborator

@wprins-JNPR wprins-JNPR left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Collaborator

@jvander-JNPR jvander-JNPR left a comment

Choose a reason for hiding this comment

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

lgtm

@Kevin-DeJong Kevin-DeJong merged commit 9a8adf7 into CPT_Terraform_Dev_2 Oct 20, 2025
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.

4 participants