Skip to content

Commit a3f1aa9

Browse files
authored
Merge pull request #355 from vkucera/tools
Tools: Add more references on naming and formatting
2 parents d7a65fc + 54e3778 commit a3f1aa9

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

docs/tools/README.md

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,11 @@ To use Clang-Tidy, you need to have O2Physics compiled and a valid symbolic link
5757

5858
### Checking naming conventions
5959

60-
The [`readability-identifier-naming`](https://clang.llvm.org/extra/clang-tidy/checks/readability/identifier-naming.html) check can fix deviations from the [naming conventions](https://rawgit.com/AliceO2Group/CodingGuidelines/master/naming_formatting.html).
60+
The [`readability-identifier-naming`](https://clang.llvm.org/extra/clang-tidy/checks/readability/identifier-naming.html) check can fix deviations from the [naming conventions](https://rawgit.com/AliceO2Group/CodingGuidelines/master/naming_formatting.html) using the configuration in [`.clang-tidy`](https://github.com/AliceO2Group/O2Physics/blob/master/.clang-tidy).
61+
62+
```tip
63+
Learn how to form a [correct camelCase name](https://google.github.io/styleguide/javaguide.html#s5.3-camel-case).
64+
```
6165

6266
### Cleaning `#include` statements and `using` statements
6367

@@ -95,12 +99,16 @@ See also the [C++ Core Guidelines](https://isocpp.github.io/CppCoreGuidelines/Cp
9599

96100
The [`format_includes.awk`](https://github.com/AliceO2Group/O2Physics/blob/master/Scripts/format_includes.awk) script allows to fix the include format in a provided O2Physics file.
97101

98-
To fix the include format in all `.h`, `.cxx` files in the current directory (`.`), execute:
102+
To fix the include format in all `.h`, `.cxx` files in O2Physics, execute:
99103

100104
```bash
101105
find . -name "*.h" -o -name "*.cxx" | parallel "awk -f Scripts/format_includes.awk \"{}\" > \"{}.tmp\" && mv \"{}.tmp\" \"{}\""
102106
```
103107

108+
```note
109+
The include format is corrected automatically in O2Physics PRs as part of the automatic formatting PR created by the [clang-format CI check](../gettingstarted/contributingtocode.md#automatic-checks).
110+
```
111+
104112
## [Cppcheck](https://cppcheck.sourceforge.io/)
105113

106114
Cppcheck is a static analysis tool for C/C++ code that detects bugs, undefined behaviour, and dangerous coding constructs that compilers typically miss.

0 commit comments

Comments
 (0)