Skip to content

Add clang-format rules to enforce include order rules#544

Merged
vrn-sn merged 4 commits intoprimaryfrom
vrn-sn/include-rules
Nov 8, 2025
Merged

Add clang-format rules to enforce include order rules#544
vrn-sn merged 4 commits intoprimaryfrom
vrn-sn/include-rules

Conversation

@vrn-sn
Copy link
Copy Markdown
Member

@vrn-sn vrn-sn commented Nov 8, 2025

Changes are in .clang-format and were applied to all our .cpp and .h files in the lute and tests directories (ignoring doctest.h).

The order is:

  1. The current source file's header.
  2. Any directory-local files like commands.h and generated/* files.
  3. All lute/ headers.
  4. All Luau/ headers.
  5. Luau VM headers (aren't namespaced).
  6. Third-party headers like uv.h, curl/, etc.
  7. Standard library headers.
  8. Other stuff that didn't get matched.

@vrn-sn vrn-sn merged commit 60db869 into primary Nov 8, 2025
11 checks passed
@vrn-sn vrn-sn deleted the vrn-sn/include-rules branch November 8, 2025 00:43
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