ILXL / ILXL/cppaudit

use meld for formatcheck

Offen
#18 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
C++
Sterne
1
Forks
1
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

As we aim to make output more readable, we want to do the same for the formatchecker. At the moment, we are using diff which may be hard to interpret.

A suggestions is to use meld instead which is a graphical diff visualizer. We can add this to the Makefile so that you have something like mformatcheck which uses meld instead of diff.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne im Makefile und suche das vorhandene formatcheck-Target sowie den formatchecker-Befehl, den es ausführt. Füge ein mformatcheck-Target hinzu oder überprüfe es, das meld anstelle von diff verwendet, und führe dann beide Targets aus, um zu bestätigen, dass der Ausgabevergleich funktioniert.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
build-system, tooling
Issue-Typ
Feature
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.