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

Disable fuchsia-related checks in clang-tidy

未关闭
#10 1 条评论 0 个 reaction 已指派 1 人 在 GitHub 查看

@BorjaOuterelo 已经在做这个了。

开始于 2019年1月16日。

enhancement
主要语言
C
星标
101
派生
35
平均合并
4 天 16 小时
30 天内合并 PR
3

描述

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?

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。