Skip to content
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

Merged
merged 10 commits into from
Mar 15, 2022
Merged

fix: change the function_grant documentation example privilege to usage #901

merged 10 commits into from
Mar 15, 2022

Conversation

jrobison-sb
Copy link
Contributor

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.

@jrobison-sb jrobison-sb changed the title [FIX-FUNCTION] change the function_grant documentation example privilege to usage fix: change the function_grant documentation example privilege to usage Mar 10, 2022
@alldoami
Copy link
Contributor

alldoami commented Mar 11, 2022

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

@jrobison-sb
Copy link
Contributor Author

Oops, thanks for correcting me. Have done so.

@alldoami
Copy link
Contributor

Sorry, @jrobison-sb you'll also need to run make docs

…thub.com:jrobison-sb/terraform-provider-snowflake into fix-function-grant-privilege-docs
@jrobison-sb
Copy link
Contributor Author

@alldoami thanks, have done so.

@alldoami
Copy link
Contributor

/ok-to-test sha=0ec2d8c

@github-actions
Copy link

Integration tests success for 0ec2d8c

@Snowflake-Labs Snowflake-Labs deleted a comment from github-actions bot Mar 15, 2022
@alldoami
Copy link
Contributor

/ok-to-test sha=fe75bf5

@github-actions
Copy link

Integration tests success for fe75bf5

@alldoami alldoami merged commit 70d9550 into Snowflake-Labs:main Mar 15, 2022
daniepett pushed a commit to daniepett/terraform-provider-snowflake that referenced this pull request Mar 22, 2022
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.

None yet

2 participants