Skip to content

Commit 66c8eb8

Browse files
add final touches
1 parent 7be46b9 commit 66c8eb8

File tree

2 files changed

+21
-1
lines changed

2 files changed

+21
-1
lines changed

.clang-format

+1
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ IndentCaseLabels: 'true'
2020
IndentWidth: '2'
2121
NamespaceIndentation: All
2222
PointerAlignment: Left
23+
SortIncludes: SI_Never
2324
SpaceAfterCStyleCast: 'true'
2425
SpaceAfterLogicalNot: 'false'
2526
SpaceAfterTemplateKeyword: 'false'

README.md

+20-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,27 @@ target_link_libraries(my_app PRIVATE ReSolve::ReSolve)
5353

5454
For all contributions to ReSolve please follow the [developer guidelines](CONTRIBUTING.md)
5555

56+
### Pre-commit
57+
Precommit is a framework for mmangaing and maintain multi-lanagueg pre-commit hooks. https://pre-commit.com/
5658

59+
Precommit is auto applied through CI to every pull request and every commit to the said pull request. Precommit checks for valid yaml files, end of file spaces, Syntax of C codeand etc. To see specifically what formating precommit looks for please check out our .clang-format and .cmake-format filed as well as our .pre-commit-config.yaml which show the specific precommit hooks being applied.
60+
61+
### Running Pre-commit locally
62+
To run pre-commit locally you first must install pre-commit. It can be installed via
63+
```shell
64+
pip install pre-commit
65+
```
66+
67+
To install the hooks and have pre-commit run automatically before every commit run
68+
```shell
69+
pre-commit install
70+
```
71+
72+
If you want to manually run all pre-commit hooks on a repository, run
73+
```shell
74+
pre-commit run --all-files
75+
```
76+
To learn more about pre-commit usuage check out https://pre-commit.com/#usage.
5777

5878
## Test and Deploy
5979

@@ -111,4 +131,3 @@ contributions to ReSolve must be made under the smae licensing terms.
111131
**Please Note** If you are using ReSolve with any third party libraries linked
112132
in (e.g., KLU), be sure to review the respective license of the package as that
113133
license may have more restrictive terms than the ReSolve license.
114-

0 commit comments

Comments
 (0)