-
Notifications
You must be signed in to change notification settings - Fork 427
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: change the function_grant documentation example privilege to usage #901
fix: change the function_grant documentation example privilege to usage #901
Conversation
You'll need to add comment changes to the schema itself and the example tf file, not the md. The md is auto-generated using a tf docs plugin. Please add changes here: https://github.com/chanzuckerberg/terraform-provider-snowflake/blob/main/pkg/resources/function_grant.go#L65 and https://github.com/chanzuckerberg/terraform-provider-snowflake/blob/main/examples/resources/snowflake_function_grant/resource.tf#L18 |
Oops, thanks for correcting me. Have done so. |
Sorry, @jrobison-sb you'll also need to run |
…thub.com:jrobison-sb/terraform-provider-snowflake into fix-function-grant-privilege-docs
@alldoami thanks, have done so. |
/ok-to-test sha=0ec2d8c |
Integration tests success for 0ec2d8c |
/ok-to-test sha=fe75bf5 |
Integration tests success for fe75bf5 |
The docs for
snowflake_function_grant
specify a privilege that you can't actually use. This changes the doc to use a valid privilege.Test Plan
N/A, documentation only.