Skip to content
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

Take off hcn package CODEOWNERS line #1017

Merged
merged 1 commit into from
May 6, 2021

Conversation

dcantah
Copy link
Contributor

@dcantah dcantah commented May 5, 2021

The CODEOWNERS logic for the hcn package shouldn't work as the required reviewer
doesn't have write access which is required.

From https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners:

"The people you choose as code owners must have write permissions for the repository. When the code owner
is a team, that team must have write permissions, even if all the individual members of the team already have
write permissions directly, through organization membership, or through another team membership."

Signed-off-by: Daniel Canter dcanter@microsoft.com

The CODEOWNERS logic for the hcn package shouldn't work as the required reviewer
doesn't have write access which is required.

From https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners:

"The people you choose as code owners must have write permissions for the repository. When the code owner
is a team, that team must have write permissions, even if all the individual members of the team already have
write permissions directly, through organization membership, or through another team membership."

Signed-off-by: Daniel Canter <dcanter@microsoft.com>
@dcantah dcantah requested a review from a team as a code owner May 5, 2021 00:40
@dcantah
Copy link
Contributor Author

dcantah commented May 5, 2021

@erfrimod Had mentioned setting up a Github team for the reviewers of any of the hcn stuff but the same applies, the team would need write access here.

@cby-in-ict
Copy link

The CODEOWNERS logic for the hcn package shouldn't work as the required reviewer
doesn't have write access which is required.

From https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners:

"The people you choose as code owners must have write permissions for the repository. When the code owner
is a team, that team must have write permissions, even if all the individual members of the team already have
write permissions directly, through organization membership, or through another team membership."

Signed-off-by: Daniel Canter dcanter@microsoft.com

Hi,sorry to bother you. I am a student for WOCW,I have a simple question.
How can I use it to manage or "docker update" for windows container which created by Docker Desktop in Windows?
I am refering Kathryn Baldauf's branch now, but don't realy understand how to use it to update a running Windows Container resource.

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

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