-
Notifications
You must be signed in to change notification settings - Fork 107
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
feat: add v4 branch protections #128
Conversation
1c19047
to
4a001f6
Compare
This appears to need a rebase. |
@taharah do you have time to get this ready for review? This would be helpful for my team. If now I’m happy to take it over. |
Also - optional attrs came out of preview so that part can be removed! (Min tf version will need to be bumped up) |
I'll get this updated tonight for TF 1.3+. On another note, should this continue to maintain backwards compatibility for v3 protections, or would a breaking change with the appropriate version bump be acceptable? |
Signed-off-by: Trevor Wood <[email protected]>
4d74080
to
d17416d
Compare
Signed-off-by: Trevor Wood <[email protected]>
I am not a maintainer of this repo, though I'm happy to give it a second set of eyes. |
I'm not a maintainer either. Only a happy user, and hopefully also able to help to get this into the main branch. |
hey guys, sorry for the delay! We'll review this shortly and come back to you! |
Signed-off-by: Trevor Wood <[email protected]>
@mariux or @soerenmartius would it be possible to get a review please? |
sorry for the delay... we added the provided feature but in a slightly different way to support computed values in some scenarios also restoring terraform 1.0 compatibility. We just released the new version. Thanks again for providing the feature. |
Adds v4 branch protections as a separate resource. If v4 branch protections are set, then it will disable the v3 branch protection resource to prevent any conflicts.
Resolves #124
Signed-off-by: Trevor Wood [email protected]