-
Notifications
You must be signed in to change notification settings - Fork 418
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
warehouse_grant: update warehouse grants in-place instead of recreating #478
Conversation
/ok-to-test sha=31c2162 |
Integration tests success for 31c2162 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also write a test for Update here? Same goes for the other resources you opened a PR for 😄 thank you!
Could you point me to any |
I know where to put the tests, I was expecting a sample resource update-grant test :) |
@igungor ah sorry, we don't have any example tests right now, would you need help writing them? |
/ok-to-test sha=020a98d |
/ok-to-test sha=b6582a5 |
Integration tests success for 020a98d |
Integration tests success for b6582a5 |
Related to #469
Updates #137