Disable fuchsia-related checks in clang-tidy

Open
#10 1 comment 0 reactions 1 assignee View on GitHub

@BorjaOuterelo is already working on this.

Since Jan 16, 2019.

Assessment

This issue has not been assessed yet.

Description

enhancement

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?

Dominant language
C
Stars
101
Forks
35
Avg merge
4d 16h
Merged PRs (30d)
3

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from micro-ROS/micro-ROS-demos

All issues in micro-ROS/micro-ROS-demos

Similar issues

More C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.