ILXL / ILXL/cppaudit

use meld for formatcheck

Ouverte
#18 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
C++
Étoiles
1
Forks
1
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez dans le Makefile et localisez la target formatcheck existante ainsi que la commande formatchecker qu’elle exécute. Ajoutez ou vérifiez une target mformatcheck qui utilise meld à la place de diff, puis exécutez les deux targets pour confirmer que la comparaison de la sortie fonctionne.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Domaine
build-system, tooling
Type d'issue
Fonctionnalité
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.