search

solvcon / modmesh

Toolkit for solving partial differential equations
BSD 3-Clause "New" or "Revised" License
29 stars 35 forks source link

Disable clang-format reordering includes #351

Closed tigercosmos closed 1 week ago

tigercosmos commented 4 weeks ago

Prevent clang-format reordering the includes.

see discussion here: https://github.com/solvcon/modmesh/pull/349#discussion_r1629451567

tigercosmos commented 1 week ago

Actually, we already defined it https://github.com/solvcon/modmesh/blob/ac4cd11dc9fc26d383cb41ab1646255516d39727/.clang-format#L135

I think it's my VSCode issue, seems the extension just ignores that flag.

yungyuc commented 1 week ago

I think it's my VSCode issue, seems the extension just ignores that flag.

This is a good finding and we can point others to your comment