SIGN IN SIGN UP

chore: add clang tidy (#1920)

* style: fix include order

* chore: add clang-tidy config

* ci: add clang tidy job to lint workflow

* refactor: fix some clang-tidy warnings

* chore: ban Qt module includes

* ci: fix clang-tidy source filter

* refactor: avoid std::endl in version helper

* chore: disable redundant access specifiers clang-tidy

Qt slots: false positive

* refactor(services): use override for dtors

* refactor: use std::min/std::max instead of if

* refactor: drop redundant casts

* refactor: use Q_UNUSED instead of unnamed params

* refactor: fix inconsistent param names

* chore: disable implicit bool conversion warning

* refactor: fix more clang-tidy warnings

* refactor: fix even more warnings

* refactor: use auto and braced init list returns

* chore: remove ctor init list warning

* refactor(settings): prefer member init list

* refactor: use [[nodiscard]]

* refactor: no inline method definitions

* refactor: fix const correctness warnings

* refactor: fix more clang-tidy warnings

* refactor: fix performance move arg warning

* refactor: use anyof/allof instead of manual loops

* perf: use uint8 enums instead of int enums

* refactor: fix use static warning

* refactor: use std::ranges where applicable

* refactor: pass some params by value & move them

* refactor: use designated struct initialisers

* perf: pass args by const ref where applicable

* chore: disable macro paren warning

* refactor: use default vals instead of overloads

* chore: disable multi inherit warning

It is required for QQmlParserStatus

* refactor: use same access modifier as base method

* fix(config): use virtual font bind method

* refactor: fix some naming convention violations

* refactor: remove unused headers

* chore: disable clang-tidy include cleaner

* refactor: fix constants naming convention violations

* chore: disable analyser false positives

* chore: ignore macros in function complexity

* refactor(core): reuse qalculator eval for async eval

It was basically the same but returned a pair instead of a single
message
Also fixes the cognitive complexity warning

* refactor(services): reduce sensorslib function complexity

* refactor(core): reduce CUtils::saveItem complexity

* refactor(services): split Storage::tick into more methods

* refactor(models): split fs model updateEntriesForDir

* refactor(blobs): reduce method complexity

Also cap cached rects at 16 to avoid UB when over 32 (they are capped at
16 when sent to the GPU anyways)

* refactor(components): improve lazy list view code quality

* ci: configure cmake to export compile commands
2
2 * r + 2 * t committed
5f37c31e40324a9a1125831e5a67e9ca0c76b9b4
Parent: d205443
Committed by GitHub <noreply@github.com> on 8/31/2026, 3:19:41 PM