daphne-project/daphne

Formatting in tests

Offen

#849 geöffnet am 12.10.2024

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (84 Forks)auto 404
Testsgood first issue

Repository-Metriken

Stars
 (80 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Since we added the .clang-format file, some of the tests that were formatted look a bit off now. For example, the brace initialized std::vector here are no longer formatted as a 4x6 matrix.

One possible solution is to check out the clang-format settings and figure out if there's a setting that works for our test code.

https://clang.llvm.org/docs/ClangFormatStyleOptions.html

Another one would be to put // clang-format [off | on] around such sections we do not want to have formatted.

Contributor Guide

Research-Richtung
Überprüfen Sie die aktuelle .clang format Konfiguration und testen Sie die Auswirkungen verschiedener Optionen auf die spezifische Testdatei. Alternativ schlagen Sie vor, clang format off/on Kommentare um die Matrixinitialisierungen herum hinzuzufügen.
Tech Stack
cpp
Domain
developer experience
Issue Type
Wartung
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80