libgit2 / libgit2/libgit2sharp
Throw when xxxOptions are null
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C#
- Sterne
- 3.5k
- Forks
- 925
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
From https://github.com/libgit2/libgit2sharp/pull/1068/files#r31650592
We rely on the following pattern throughout the codebase.
public Things DoThings()
{
return DoThings(null);
}
public Things DoThings(ThingsOptions options)
{
options = options ?? new ThingsOptions();
[...]
}
Let's change this to the following one
public Things DoThings()
{
return DoThings(new ThingsOptions());
}
public Things DoThings(ThingsOptions options)
{
Ensure.ArgumentNotNull(options);
[...]
}
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Durchsuche die C#-Codebasis nach Überladungen, die xxxOptions akzeptieren und derzeit null durch eine neue Optionsinstanz ersetzen. Lies zuerst die Diskussion im referenzierten Pull Request und ermittle anschließend jede betroffene Überladung sowie ihr parameterloses Gegenstück. Als erledigt gilt die Aufgabe, wenn diese Überladungen null-Optionen ablehnen und parameterlose Aufrufe das Standard-Optionsobjekt erzeugen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- csharp
- Bereich
- backend-api-design
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100