This repository was archived by the owner on Dec 3, 2023. It is now read-only.

Conversation

BenWhitehead

According to https://cloud.google.com/iam/docs/conditions-overview#syntax_overview both title and description are optional.

Currently, AutoValue treats these fields as non-null which goes against the definition from the API.

This change may be considered breaking by some criteria, however this class is annotated @BetaApi.

@product-auto-labelproduct-auto-label bot added the size: sPull request size is small.label Jan 3, 2023
@BenWhiteheadBenWhitehead force-pushed the iam-policy-condition-fix branch from d749b01 to 419a713 Compare January 3, 2023 19:34
According to https://cloud.google.com/iam/docs/conditions-overview#syntax_overview both title and description are optional.

Currently, AutoValue treats these fields as non-null which goes against the definition from the API.

This change may be considered breaking by some criteria, however this class is annotated @BetaApi.
@BenWhiteheadBenWhitehead force-pushed the iam-policy-condition-fix branch from 419a713 to 44ce24a Compare January 3, 2023 20:29
@BenWhiteheadBenWhitehead added the owlbot:runAdd this label to trigger the Owlbot post processor.label Jan 3, 2023
@gcf-owl-botgcf-owl-bot bot removed the owlbot:runAdd this label to trigger the Owlbot post processor.label Jan 3, 2023
@BenWhiteheadBenWhitehead marked this pull request as ready for review January 3, 2023 21:27
@BenWhiteheadBenWhitehead requested a review from a team as a code owner January 3, 2023 21:27
@BenWhiteheadBenWhitehead merged commit 1238462 into main Jan 3, 2023
@BenWhiteheadBenWhitehead deleted the iam-policy-condition-fix branch January 3, 2023 22:00
@release-pleaserelease-please bot mentioned this pull request Jan 3, 2023
Sign up for free to subscribe to this conversation on . Already have an account? Sign in.
size: sPull request size is small.
None yet

Successfully merging this pull request may close these issues.