Skip to content

Commit 2d50135

Browse files
committed
apply prettier; remove style check
1 parent 0b93919 commit 2d50135

File tree

4 files changed

+19
-21
lines changed

4 files changed

+19
-21
lines changed

.clang-format

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
11
---
2-
BasedOnStyle: Google
3-
AccessModifierOffset: '-2'
4-
AlignTrailingComments: 'true'
5-
AllowAllParametersOfDeclarationOnNextLine: 'false'
6-
AlwaysBreakTemplateDeclarations: 'No'
7-
BreakBeforeBraces: Attach
8-
ColumnLimit: '100'
9-
ConstructorInitializerAllOnOneLineOrOnePerLine: 'true'
10-
IncludeBlocks: Regroup
11-
IndentPPDirectives: AfterHash
12-
IndentWidth: '4'
13-
NamespaceIndentation: All
14-
BreakBeforeBinaryOperators: All
15-
BreakBeforeTernaryOperators: 'true'
16-
...
2+
BasedOnStyle: Google
3+
AccessModifierOffset: "-2"
4+
AlignTrailingComments: "true"
5+
AllowAllParametersOfDeclarationOnNextLine: "false"
6+
AlwaysBreakTemplateDeclarations: "No"
7+
BreakBeforeBraces: Attach
8+
ColumnLimit: "100"
9+
ConstructorInitializerAllOnOneLineOrOnePerLine: "true"
10+
IncludeBlocks: Regroup
11+
IndentPPDirectives: AfterHash
12+
IndentWidth: "4"
13+
NamespaceIndentation: All
14+
BreakBeforeBinaryOperators: All
15+
BreakBeforeTernaryOperators: "true"

README.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
# py2cpp
1313

14-
1514
## ✨ Features
1615

1716
- [Modern CMake practices](https://pabloariasal.github.io/2018/02/19/its-time-to-do-cmake-right/)
@@ -42,7 +41,7 @@
4241
Eventually, you can remove any unused files, such as the standalone directory or irrelevant github workflows for your project.
4342
Feel free to replace the License with one suited for your project.
4443

45-
To cleanly separate the library and subproject code, the outer `CMakeList.txt` only defines the library itself while the tests and other subprojects are self-contained in their own directories.
44+
To cleanly separate the library and subproject code, the outer `CMakeList.txt` only defines the library itself while the tests and other subprojects are self-contained in their own directories.
4645
During development it is usually convenient to [build all subprojects at once](#build-everything-at-once).
4746

4847
### Build and run the standalone target
@@ -64,7 +63,7 @@ cmake -S test -B build/test
6463
cmake --build build/test
6564
CTEST_OUTPUT_ON_FAILURE=1 cmake --build build/test --target test
6665

67-
# or simply call the executable:
66+
# or simply call the executable:
6867
./build/test/Py2CppTests
6968
```
7069

@@ -144,7 +143,7 @@ Additional arguments can be passed to the analyzers by setting the `CLANG_TIDY_A
144143

145144
Ccache can be enabled by configuring with `-DUSE_CCACHE=<ON | OFF>`.
146145

147-
## FAQ
146+
## FAQ
148147

149148
> Can I use this for header-only libraries?
150149
@@ -165,7 +164,7 @@ Use this as the main directory for best IDE support.
165164
> I see you are using `GLOB` to add source files in CMakeLists.txt. Isn't that evil?
166165
167166
Glob is considered bad because any changes to the source file structure [might not be automatically caught](https://cmake.org/cmake/help/latest/command/file.html#filesystem) by CMake's builders and you will need to manually invoke CMake on changes.
168-
I personally prefer the `GLOB` solution for its simplicity, but feel free to change it to explicitly listing sources.
167+
I personally prefer the `GLOB` solution for its simplicity, but feel free to change it to explicitly listing sources.
169168

170169
> I want create additional targets that depend on my library. Should I modify the main CMakeLists to include them?
171170

codecov.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ ignore:
22
- "test"
33

44
comment:
5-
require_changes: true
5+
require_changes: true

0 commit comments

Comments
 (0)