micro-ROS / micro-ROS/micro-ROS-demos

Disable fuchsia-related checks in clang-tidy

Ouverte
#10 1 commentaire 0 réactions 1 personne assignée Voir sur GitHub

@BorjaOuterelo y travaille déjà.

Depuis le 16/1/2019.

enhancement
Langage dominant
C
Étoiles
101
Forks
35
Merge moyen
4 j 16 h
PR mergées (30 j)
3

Description

Compiling the demos leads to a lot of warnings from clang-tidy, many of which are from the Fuchsia warning set, such as "fuchsia-default-arguments". Fuchsia has a very specific style guide which is not generally appropriate, e.g., calling functions with default arguments is valid C++ practice, but disallowed by Fuchsia.

I would suggest the following list instead of "" as used currently:
bugprone-
,cert-,clang-analyzer-,cppcoreguidelines-,google-,hicpp-,llvm-,misc-,modernize-,performance-,readability-

Opinions?

Guide de contribution

Ouvrir le guide de contribution

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.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

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