You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
However, the standard seems to be that s2n-tls will manually review all clang-format based changes. With the 243 non-clang formatted files, that would take quite a long time to review. I'm generally comfortable with automated contributions/tooling, but if we don't want to do that then this will need to be split into multiple PRs.
Requirements / Acceptance Criteria:
All .c and .h files must be clang formatted.
Out of scope:
Nothing is out of scope. All .c. and .h files in the repo must be clang formatted.
The text was updated successfully, but these errors were encountered:
Problem:
Not all files are clang formatted. The missed files seem to be
Solution:
However, the standard seems to be that s2n-tls will manually review all clang-format based changes. With the 243 non-clang formatted files, that would take quite a long time to review. I'm generally comfortable with automated contributions/tooling, but if we don't want to do that then this will need to be split into multiple PRs.
Requirements / Acceptance Criteria:
All
.c
and.h
files must be clang formatted.Out of scope:
Nothing is out of scope. All
.c.
and.h
files in the repo must be clang formatted.The text was updated successfully, but these errors were encountered: