dotnet/runtime

Respond to PR code review comments for ported library System.Security.AccessControl

Fechada

#15.934 aberto em 14 de dez. de 2015

Ver no GitHubGuia do colaborador
 (9 comentários) (0 reação) (0 responsável)C# (5.445 forks)batch import
area-System.Securityenhancementhelp wanted

Métricas do repositório

Stars
 (17.886 estrelas)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

Description

@GiriSr as the owner of the System.Security.AcessControl library can you please have a look through the code review comments provided in https://github.com/dotnet/corefx/pull/4977 and address them appropriately.

Guia do colaborador

Direção de pesquisa
Revise os comentários da revisão de código no PR vinculado (dotnet/corefx#4977) para entender o feedback. Verifique se a biblioteca System.Security.AccessControl portada no dotnet/runtime já abordou esses comentários ou se eles ainda não foram resolvidos. Determine a resposta apropriada para cada comentário, o que pode incluir fazer alterações no código ou fornecer explicações. Consulte o mantenedor (@GiriSr) se necessário para esclarecimentos.
Pilha de tecnologia
csharp
Domain
backendsecurity
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#System.Security.AccessControlCode review
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
25