Skip to content

Conversation

@DanAlbert
Copy link
Member

This used to be done by CI but we ripped that out since there wasn't any good way to ensure that CI and and local workflows were using the same clang-format. Back when it was in CI, the pointer alignment was configured incorrectly (we used the Google style presets, but those are alignment preserving rather than actually style enforcing). Reformat everything since the .clang-format file has changed since then so I stop including so many unrelated edits in my commits.

This used to be done by CI but we ripped that out since there wasn't any
good way to ensure that CI and and local workflows were using the same
clang-format. Back when it was in CI, the pointer alignment was
configured incorrectly (we used the Google style presets, but those are
alignment preserving rather than actually style enforcing). Reformat
everything since the .clang-format file has changed since then so I stop
including so many unrelated edits in my commits.
@DanAlbert DanAlbert requested a review from enh-google August 26, 2025 22:50
@DanAlbert DanAlbert merged commit 5c283e2 into android:main Aug 27, 2025
2 checks passed
@DanAlbert DanAlbert deleted the clang-format-everything branch August 27, 2025 17:51
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