Skip to content

Include "C API" files in clang-format check#151

Merged
chinmayshah99 merged 2 commits intoOpenMined:devfrom
replomancer:clang_format_check_fix
May 22, 2020
Merged

Include "C API" files in clang-format check#151
chinmayshah99 merged 2 commits intoOpenMined:devfrom
replomancer:clang_format_check_fix

Conversation

@replomancer
Copy link
Copy Markdown
Member

@replomancer replomancer commented May 20, 2020

Description

Apparently files in src/bindings/c are excluded from the style check and do not follow the formatting standard. Some other files avoided the check probably because of missing -o in the find command (details are in this PR style check results).

This PR only fixes the style check (no files reformatted yet).

@chinmayshah99
Copy link
Copy Markdown
Member

@replomancer Can we format these C API files using clang-format and add them to this PR?

@chinmayshah99 chinmayshah99 merged commit 6f2b947 into OpenMined:dev May 22, 2020
dvadym pushed a commit to dvadym/PyDP that referenced this pull request Jul 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants