Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Formatted headers and added style check. #212

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Formatted headers and added style check. #212

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6933208
Formatted headers and added style check.
Kerilk Oct 5, 2022
2bcd9b6
Remove clang format version.
Kerilk Oct 5, 2022
34757ee
Formatted c files for CI simplicity.
Kerilk Oct 5, 2022
1bdddde
Force style check on push, and fix windows header order.
Kerilk Oct 5, 2022
cc767a6
Simplify style check: run on all code base.
Kerilk Oct 5, 2022
215b3be
Fix style check.
Kerilk Oct 5, 2022
13085ac
Force recent ubuntu image for updated clang-format.
Kerilk Oct 5, 2022
f77feab
Fix missing files.
Kerilk Oct 5, 2022
e7b546f
Simplified .clang-format to better account for differences vs base st…
Kerilk Oct 5, 2022
648d170
Indent to 4 spaces as in the headers.
Kerilk Oct 5, 2022
5bfd3d5
Use pointer alignement middle.
Kerilk Oct 5, 2022
a1565f6
Change indent size and function argument breaks.
Kerilk Oct 5, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions .clang-format
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
---
BasedOnStyle: Mozilla
Language: Cpp
ColumnLimit: 100
AlignAfterOpenBracket: AlwaysBreak
IndentWidth: 4
ContinuationIndentWidth: 4
PointerAlignment: Middle
AlignConsecutiveMacros: AcrossEmptyLinesAndComments
AlignConsecutiveDeclarations: AcrossEmptyLinesAndComments
SortIncludes: false
BraceWrapping:
AfterFunction: true
BreakBeforeBraces: Custom

15 changes: 15 additions & 0 deletions .github/workflows/style.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
name: style-checks
on: [push, pull_request]
jobs:
clang-format:
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v2
- run: sudo apt-get update
- run: sudo apt-get install -y clang-format
- name: run git-clang-format
run: clang-format --style=file --dry-run --Werror `git ls-files *.c *.h` >clang-format-diff 2>&1
shell: bash
- name: output clang format diff
if: always()
run: cat ./clang-format-diff
Loading