Skip to content

Conversation

@SeanTAllen
Copy link
Contributor

This includes a small switch from the first version of policy tool that had
command as a string. The single string was problematic as that isn't the
representation in gcs. I updated the representation to match as part of this PR.

… containers

This includes a small switch from the first version of policy tool that had
command as a string. The single string was problematic as that isn't the
representation in gcs. I updated the representation to match as part of this PR.
@SeanTAllen SeanTAllen requested a review from a team as a code owner August 16, 2021 13:28
Copy link
Contributor

@anmaxvl anmaxvl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@SeanTAllen
Copy link
Contributor Author

@anmaxvl do we need additional reviews or can this be merged?

@anmaxvl
Copy link
Contributor

anmaxvl commented Aug 21, 2021

@anmaxvl do we need additional reviews or can this be merged?

need one more +1 😄 @microsoft/containerplat, PTAL

Copy link
Contributor

@msscotb msscotb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@SeanTAllen
Copy link
Contributor Author

I got two approvals, can I get a merge?

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.

3 participants