chore: fix all clang-tidy findings in the existing code - #86
Open
EnriqueParodi wants to merge 1 commit into
Open
chore: fix all clang-tidy findings in the existing code#86EnriqueParodi wants to merge 1 commit into
EnriqueParodi wants to merge 1 commit into
Conversation
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
2 times, most recently
from
August 18, 2026 23:01
2a4b650 to
28f1641
Compare
EnriqueParodi
changed the base branch from
feat/web-explorer
to
ci/release-changelog
August 18, 2026 23:15
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
2 times, most recently
from
August 18, 2026 23:19
e8d8ab2 to
eb97dcf
Compare
EnriqueParodi
force-pushed
the
ci/release-changelog
branch
from
August 18, 2026 23:47
d8d4136 to
4ca843b
Compare
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
from
August 18, 2026 23:47
eb97dcf to
70d99da
Compare
EnriqueParodi
marked this pull request as ready for review
August 18, 2026 23:49
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
from
August 19, 2026 07:42
70d99da to
25add43
Compare
This was referenced Aug 19, 2026
EnriqueParodi
force-pushed
the
ci/release-changelog
branch
from
August 19, 2026 12:18
4ca843b to
3c4dd44
Compare
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
from
August 19, 2026 12:18
25add43 to
f5c06f3
Compare
EnriqueParodi
force-pushed
the
ci/release-changelog
branch
from
August 19, 2026 12:41
3c4dd44 to
a823e6d
Compare
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
from
August 19, 2026 12:41
f5c06f3 to
a8ab214
Compare
EnriqueParodi
force-pushed
the
ci/release-changelog
branch
from
August 19, 2026 21:21
a823e6d to
7992e02
Compare
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
2 times, most recently
from
August 20, 2026 11:26
35f6af8 to
28f731c
Compare
EnriqueParodi
force-pushed
the
ci/release-changelog
branch
from
August 20, 2026 11:26
7992e02 to
083b945
Compare
The new nightly clang-tidy job runs over the whole tree; this clears every finding its first full run reported, so the job is green from night one. Where a check is wrong or the code is deliberate, the waiver is written next to the code together with its reason. It also carries the unused variable removal from #63, which gcc-16 reports as an error. Co-authored-by: Florian Sattler <florian.sattler@airbus.com>
EnriqueParodi
force-pushed
the
ci/release-changelog
branch
from
August 20, 2026 21:03
083b945 to
1993a18
Compare
EnriqueParodi
force-pushed
the
chore/clang-tidy-main
branch
from
August 20, 2026 21:03
28f731c to
b4db22e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The new nightly clang-tidy job runs over the whole tree; this clears every
finding its first full run reported, so the job is green from night one.
Where a check is wrong or the code is deliberate, the waiver is written
next to the code together with its reason.