-
Notifications
You must be signed in to change notification settings - Fork 929
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
Add approval_rules in protected_environments #1727
Add approval_rules in protected_environments #1727
Conversation
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.
Few small comments, but nothing serious...
protected_environments_test.go
Outdated
"required_approvals": 2, | ||
"group_inheritance_type": 1 | ||
} | ||
] |
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.
I looks like your indention is incorrect here...
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.
Well spotted, probably the paste from the official doc that went wrong.
@svanharmelen thanks for the quick review; I addressed the comments, let me know if you spot anything else! |
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.
I made a few last changes (the indentation was a space vs tab issue), things are OK now 👍🏻
Aaaah yes that was it, thanks for spotting it, I'll need to adjust my editor. |
Coming from here: https://gitlab.com/gitlab-org/terraform-provider-gitlab/-/issues/1417