eclipse-iceoryx / eclipse-iceoryx/iceoryx

Cleanup superfluous `clang-tidy` suppressions

Open
#1,615 0 comments 0 reactions 0 assignees View on GitHub
clang-tidy globex technical debt
Dominant language
C++
Stars
2.2k
Forks
492
Avg merge
18h 57m
Merged PRs (30d)
1

Description

## Brief feature description

In some parts of the code certain `clang-tidy` rules are suppressed which are also deactivated in the `.clang-tidy` file. Those suppression should be removed from the code.

Furthermore, a CI script shall be written which verifies that no new suppression's for rules which are deactived are introduced.
The CI shall fail and block the code when this happens.

Extracted from: #1196

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the repository's .clang-tidy file and locating code suppressions for rules disabled there. Then inspect the existing CI configuration to determine where a verification script belongs. Done means redundant suppressions are removed and CI blocks new suppressions for deactivated rules.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
ci-cd, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.