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.

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

clang-format-15 for all remaining .C .cxx .h .hh #857

Closed
wants to merge 1 commit into from

Conversation

inkdot7
Copy link
Contributor

@inkdot7 inkdot7 commented May 6, 2023

Changes needed due to #828 for other merge requests to pass CI format checks.

I have not reviewed them carefully, just slightly with git diff --word-diff. I believe review is the responsibility proponents of clang-format-15.

@inkdot7 inkdot7 force-pushed the clang-format-15-all branch from 46896ca to 69333d7 Compare May 6, 2023 17:20
@inkdot7
Copy link
Contributor Author

inkdot7 commented May 6, 2023

The push of _LAYOUT_INIT removes was a push to wrong branch. Please ignore.

@inkdot7 inkdot7 mentioned this pull request May 6, 2023
@inkdot7 inkdot7 force-pushed the clang-format-15-all branch from 69333d7 to 9a0f4b5 Compare May 6, 2023 19:16
@jose-luis-rs
Copy link
Contributor

Hi @inkdot7
In my previous PRs #854 and #855 I didn't apply the clang-format-15 over all the existing files to avoid conflicts with the current PRs. This could be now the case. Please, let us do this later, in particular, after #832 and #842

@jose-luis-rs
Copy link
Contributor

This is done in the PR #1011

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