Where to include message for #pragma disable?
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 25/100
- Issue-Typ
- Dokumentation
- Klarheit
- Muss geklärt werden
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- csharp
- Bereich
- documentation
Rechercherichtung
Überprüfe die beiden C#-Beispiele im Issue und die vorhandenen Hinweise des Repositorys zu Coding-Konventionen. Lege ein einheitliches Format für Begründungsnachrichten fest, die #pragma warning disable begleiten, einschließlich Platzierung und Formulierung, und dokumentiere die Hinweise konsistent mit dem ausgewählten Beispiel.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
If I am disabling a warning, what is the format of the justification message.
Here's an idea/approach but it is still ugly, better suggestions welcome:
// Justifiction: Properties initialized by Entity Framework.
#nullable disable // CS8618: Non-nullable field is uninitialized. Consider declaring as nullable.
public ApplicationDbContext(
#nullable enable
DbContextOptions<ApplicationDbContext>? options) : base(options) { }
// Justifiction: Properties initialized by Entity Framework.
#nullable disable // CS8618: Non-nullable field is uninitialized. Consider declaring as nullable.
public ApplicationDbContext(
#nullable enable
or, with pragma:
// Justification: Uninitialized properties set by Entity Framework.
#pragma warning disable CS8618 // Non-nullable field is uninitialized. Consider declaring as nullable.
public ApplicationDbContext(DbContextOptions<ApplicationDbContext> options) : base(options) { }
#pragma warning restore CS8618 // Non-nullable field is uninitialized. Consider declaring as nullable.
- Vorherrschende Sprache
- C#
- Sterne
- 12
- Forks
- 16
- Ø Merge
- 4 Min.
- Gemergte PRs (30 T.)
- 12
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus IntelliTect/CodingGuidelines
-
Overriding ToString()Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100
IntelliTect/CodingGuidelines#290 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100
IntelliTect/CodingGuidelines#253 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Review naming recommendation for fieldsEvtl. wieder frei @MarkMichaelis hat das vor 1332 Tagen übernommen, und es ist kein Pull Request offen. Offen.editorconfig C# coding guidelines proposal
IntelliTect/CodingGuidelines#249 · 6 Kommentare · 1 Reaktion · 2 zugewiesene Personen ·
Maintainer antworten meist innerhalb von 1 Tag
-
analyzer C# coding guidelines proposal
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
IntelliTect/CodingGuidelines#231 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
IntelliTect/CodingGuidelines#230 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in IntelliTect/CodingGuidelines
Ähnliche Issues
-
VideoViewer: rotated (portrait phone) videos shown sideways when system decimal separator is a commaOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Volodymyr-Petrunin/Bankomaten#45 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
AvaloniaUI/Avalonia#22420 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
dotnet/SqlClient#4823 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
-
type/automation type/tech-debt
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag