dotnet/runtime

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

Geschlossen

#15.934 geöffnet am 14.12.2015

Auf GitHub ansehenContributor Guide
 (9 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-System.Securityenhancementhelp wanted

Repository-Metriken

Stars
 (17.886 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfen Sie die Code Review Kommentare im verlinkten PR (dotnet/corefx#4977), um das Feedback zu verstehen. Prüfen Sie, ob die portierte System.Security.AccessControl Bibliothek in dotnet/runtime diese Kommentare bereits berücksichtigt hat oder ob sie noch ungelöst sind. Bestimmen Sie die angemessene Antwort für jeden Kommentar, die Codeänderungen oder Erklärungen umfassen kann. Konsultieren Sie bei Bedarf den Maintainer (@GiriSr) zur Klärung.
Tech Stack
csharp
Domain
backendsecurity
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#System.Security.AccessControlCode review
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
25