Fix for #30 - Githubissues

PaloAltoNetworks / terraform-provider-prismacloudcompute

Terraform provider for Prisma Cloud Compute

https://registry.terraform.io/providers/PaloAltoNetworks/prismacloudcompute/latest

Mozilla Public License 2.0

24 stars 28 forks source link

Fix for #30 #31

Closed hi-artem closed 2 years ago

hi-artem commented 3 years ago

Description

Motivation and Context

Look at #30

How Has This Been Tested?

I tested it locally with Prisma Cloud Compute 21.04

Types of changes

Bug fix (non-breaking change which fixes an issue)

There is a typo in provider.go preventing usage of variables in order to configure provider.

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

hi-artem commented 3 years ago

I added comments to provider example, but open to suggestions on how to better document this feature.

@wfg I would really like this to be released to Terraform registry as soon as you can. It is currently blocking us from deploying provider.

wfg commented 2 years ago

@hi-artem thanks, sorry for the delay here. I'll try to get an update pushed soon.