Skip to content

add proper include#48329

Merged
jkarneges merged 1 commit intomainfrom
jkarneges/fix-include
Apr 24, 2026
Merged

add proper include#48329
jkarneges merged 1 commit intomainfrom
jkarneges/fix-include

Conversation

@jkarneges
Copy link
Copy Markdown
Member

@jkarneges jkarneges commented Apr 23, 2026

This should be an include rather than a forward declaration, which was only working by luck. We need this for the upcoming clang-format reformatting to work, since it sorts includes.

@jkarneges jkarneges requested a review from a team April 24, 2026 00:19
@jkarneges jkarneges merged commit 01256f8 into main Apr 24, 2026
19 checks passed
@jkarneges jkarneges deleted the jkarneges/fix-include branch April 24, 2026 14:56
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