#
aeb73bc5 |
| 30-May-2025 |
Patrick Williams <patrick@stwcx.xyz> |
config-clang-tidy: add format subcommand
The 'merge' subcommand can be used for formatting, if the `--reference` is the same clang-tidy file as the input, but this is not convenient for users. Add
config-clang-tidy: add format subcommand
The 'merge' subcommand can be used for formatting, if the `--reference` is the same clang-tidy file as the input, but this is not convenient for users. Add a 'format' subcommand that does the expected.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: If51ded84e85f6f285a6b5d1df0764ff20b1b2064
show more ...
|
#
b8ce3818 |
| 30-May-2025 |
Patrick Williams <patrick@stwcx.xyz> |
config-clang-tidy: better handle wildcards
Some repositories had category wildcards; handle those better: - Remove category disables since we already disable global. - Allow category enables t
config-clang-tidy: better handle wildcards
Some repositories had category wildcards; handle those better: - Remove category disables since we already disable global. - Allow category enables to act as a single check.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ibe320717331c0627439c54846465786b9316f186
show more ...
|
#
23362867 |
| 30-May-2025 |
Patrick Williams <patrick@stwcx.xyz> |
config-clang-tidy: strip comments better
Some repositories have a Checks line with embedded comments. This is unsupported by the python YAML parser and results in strings where comment lines can no
config-clang-tidy: strip comments better
Some repositories have a Checks line with embedded comments. This is unsupported by the python YAML parser and results in strings where comment lines can no longer be identified because it becomes one big paragraph. Strip these out prior to YAML processing.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I455ea4584d71d1ff94710ad8a5c02d5eabb05b82
show more ...
|
#
b516729d |
| 23-May-2025 |
Patrick Williams <patrick@stwcx.xyz> |
config-clang-tidy: introduce tool
There have been a few efforts to enable clang-tidy across the organization, many of which have either been partially applied or inconsistently applied. Part of the
config-clang-tidy: introduce tool
There have been a few efforts to enable clang-tidy across the organization, many of which have either been partially applied or inconsistently applied. Part of the issue has been that enabling options is done by hand by humans.
Introduce a tool that allows us to more automatically manipulate the clang-tidy configs. There are currently 3 sub-commands: - merge - enable - disable
The merge command can be used to merge an existing repository clang-tidy config with a reference, such as the one in docs: ``` tools/config-clang-tidy --repo ../phosphor-virtual-sensor \ merge --reference ../docs/style/cpp/.clang-tidy ```
Similarly, the enable / disable commands can be used to enable or disable an individual check. ``` ./tools/config-clang-tidy --repo ../phosphor-virtual-sensor \ enable modernize-use-nullptr ```
This tool will enable us to create scripts jobs that: - Synchronize the clang-tidy config across a large set of repositories. - Enable a single check across a large set of repositories and examine the fallout.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ib338f7dcbc7f1049122399e0b616f77ec5eb7dc2
show more ...
|