| 18 Jul 2025 |
 emily | at least a naive "redundant include checker" would complain about including both such headers, even if it is the correct thing for portability across platforms | 03:23:59 |
 fzakaria | sure; still better than current model. You can special case the 10% rather than give up on the 90% | 15:08:38 |
 Mic92 | In reply to @fzakaria:one.ems.host
sweet -- the IWYU is what i'm really after. You mean the clang-tidy one? Works quite well on nix-eval-jobs | 17:33:14 |
| Mic92 | In reply to @qyliss:fairydust.space
IWYU is difficult for portable programs like Nix Maybe that can be enabled somehow for nix header only? | 17:34:34 |
| Mic92 | Same actually for platform specific header - could be a bit tedious though | 17:35:25 |
| Mic92 | @xokdvium:matrix.org: clang format stuff is good to go? | 17:36:58 |
 Sergei Zimmerman (xokdvium) | I think so. The diff is good and the auto-rebase script seems to work well. | 17:37:33 |
| Mic92 | Okay. Hit it! | 17:38:04 |
| Sergei Zimmerman (xokdvium) | Done! No more range formatting. Yay! | 17:40:50 |
| Sergei Zimmerman (xokdvium) | Pinned an issue about the auto-rebase script: https://github.com/NixOS/nix/issues/13502 | 18:02:18 |
