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

Disable fuchsia-related checks in clang-tidy

Offen
#10 1 Kommentar 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@BorjaOuterelo arbeitet bereits daran.

Seit 16.1.2019.

enhancement
Vorherrschende Sprache
C
Sterne
101
Forks
35
Ø Merge
4 T. 16 Std.
Gemergte PRs (30 T.)
3

Beschreibung

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?

Beitragsleitfaden

Beitragsleitfaden öffnen

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.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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