INTL0001 when using records
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
- 35/100
Rechercherichtung
Überprüfe die INTL0001-Regel und ihr Verhalten in Bezug auf attribute-guideline anhand des positional-record-Beispiels im Issue. Erledigt ist dies, wenn die Anleitung Inline-Attribute bei Record-Parametern eindeutig behandelt und die erwartete Behandlung durch den Analyzer dokumentiert und verifiziert ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
With the introduction of positional records it has become more valuable to apply attributes inline. This is especially true when dealing with serialization.
Consider the following which triggers INTL0001 (Properties must be on their own line)
public record MyDto([property:JsonPropertyName("my_Value")] string Value);
The work around is to simply do as the analyzer suggests and put it on its own line:
public record MyDto(
[property:JsonPropertyName("my_Value")]
string Value);
However, I would posit that this is less readable that the original.
We should revisit the attribute guidelines with these new use-cases in mind.
- 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
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
IntelliTect/CodingGuidelines#230 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100
IntelliTect/CodingGuidelines#229 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in IntelliTect/CodingGuidelines
Ähnliche Issues
-
area:jobads-cv BE mvp P2
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
klasolsson81/jobbliggaren#2099 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
0 - Backlog Bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
BrighterCommand/Brighter#4581 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
Esri/calcite-dotnet-toolkit#30 · 1 Reaktion ·
-
kind:docs simplification size:S status:todo
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
elsa-workflows/elsa-foundation#2604 ·
Maintainer antworten meist innerhalb von 1 Tag