dotnet/runtime

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

Chiusa

#15.934 aperta il 14 dic 2015

Vedi su GitHubGuida contributor
 (9 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-System.Securityenhancementhelp wanted

Metriche repository

Star
 (17.886 stelle)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

@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.

Guida contributor

Direzione di ricerca
Rivedi i commenti della code review nella PR collegata (dotnet/corefx#4977) per comprendere il feedback. Verifica se la libreria System.Security.AccessControl portata in dotnet/runtime ha già affrontato questi commenti o se sono ancora irrisolti. Determina la risposta appropriata per ciascun commento, che potrebbe includere apportare modifiche al codice o fornire spiegazioni. Consulta il maintainer (@GiriSr) se necessario per chiarimenti.
Tech stack
csharp
Dominio
backendsecurity
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#System.Security.AccessControlCode review
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
25