ament / ament/ament_lint

[ament_clang_format] Support pointing to config file rather than YAML load/dump for clang-format>=14.0.0

未關閉
#383 0 則留言 1 個 reaction 已指派 0 人 在 GitHub 檢視
help wanted
主要語言
Python
星號
50
分支
129
平均合併
3 天 10 小時
30 天內合併 PR
3

描述

### What?

From the [clang-format v14.0.0 changelog](https://releases.llvm.org/14.0.0/tools/clang/docs/ReleaseNotes.html#clang-format):

> The command line argument `-style=` has been extended so that a specific format file at location `` can be selected. This is supported via the syntax: `-style=file:`.

Given that the default clang-format version in Ubuntu 22.04 is [>=14~](https://packages.ubuntu.com/jammy/clang-format), it's reasonable to expect that the configuration file argument will just be passed as a file path to the invoked clang-format binary (for appropriate versions) rather than as a dumped YAML stream.

### Why?

The main use-case for this feature request is to handle cases like #364, where multiple configurations for different languages mean that there are multiple YAML documents in the same file. The YAML load API does not like this, even though this is a perfectly fine use of the clang-format configuration.

貢獻指南

開啟貢獻指南

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。