Code

Upgrade clang-tidy to version 19#166

Code

Merged

TrentHouliston merged 21 commits intomainFastcode/NUClear:mainfrom houliston/clang-tidy-19Fastcode/NUClear:houliston/clang-tidy-19Copy head branch name to clipboardDec 27, 2024

+1,128-488

Commits

Commits on Dec 25, 2024

• Cancelling should stop everything

  TrentHouliston

  committed
  fcbce01

• Upgrade to clang-tidy 19

  TrentHouliston

  committed
  2d754fd

• put everything on 24.04

  TrentHouliston

  committed
  f267881

• Turns out I want 20

  TrentHouliston

  committed
  915a4d4

• Turns out it is 19 but I'm doing it wrong?

  TrentHouliston

  committed
  9d2fa74

• Apply some fixes from the new clang-tidy

  Show description for 899a463
  TrentHouliston

  committed
  899a463

• Made no sense including nuclear as a system include in tests

  TrentHouliston

  committed
  55a9417

Commits on Dec 26, 2024

• More changes

  TrentHouliston

  committed
  e3cbbee

• nolint the bug for now, fix in another PR since it might change a bit

  TrentHouliston

  committed
  2d0cc4d

• Need the headers for windows

  TrentHouliston

  committed
  02eeaed

• Only need forward declarations here

  TrentHouliston

  committed
  20c5806

• fwd?

  TrentHouliston

  committed
  e998ffe

• Try forward declaring iovec and saying it's exported? if this doesn't work I'm just turning off the include cleaner until it's more mature

  TrentHouliston

  committed
  d824db5

• swap to a block

  TrentHouliston

  committed
  67a2840

• Don't like it

  TrentHouliston

  committed
  9c9ef96

• Should be all the includes fixed!

  TrentHouliston

  committed
  286b263

• "handle" the exception so clang-tidy is happy

  TrentHouliston

  committed
  97764fb

• invalid, clearly this is never fired

  TrentHouliston

  committed
  ff1e99b

Commits on Dec 27, 2024

• Forgot why this was important, left a comment for future me/others

  TrentHouliston

  committed
  8c41c53

• ignore .DS_Store files

  TrentHouliston

  committed
  c5e54e9

• This header shouldn't be committed

  TrentHouliston

  committed
  e0b5998