dotnet/runtime

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

Fermée

#15 934 ouverte le 14 déc. 2015

Voir sur GitHubGuide contributeur
 (9 commentaires) (0 réaction) (0 personne assignée)C# (5 445 forks)batch import
area-System.Securityenhancementhelp wanted

Métriques du dépôt

Stars
 (17 886 étoiles)
Métriques de merge PR
 (Merge moyen 12j 11h) (661 PRs mergées en 30 j)

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.

Guide contributeur

Direction de recherche
Examinez les commentaires de la revue de code dans la PR liée (dotnet/corefx#4977) pour comprendre les retours. Vérifiez si la bibliothèque System.Security.AccessControl portée dans dotnet/runtime a déjà traité ces commentaires ou s'ils restent non résolus. Déterminez la réponse appropriée pour chaque commentaire, ce qui peut inclure des modifications de code ou des explications. Consultez le mainteneur (@GiriSr) si nécessaire pour des clarifications.
Stack technique
csharp
Domaine
backendsecurity
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#System.Security.AccessControlCode review
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25