eclipse-score / eclipse-score/coverage_tool

make_clang_tidy_aspect should skip source files from external Bzlmod repositories (similar to angle_includes_are_system for headers)

Offen
#4 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Python
Sterne
0
Forks
2
Ø Merge
11 Std. 37 Min.
Gemergte PRs (30 T.)
5

Beschreibung

When a target lists a source file from an external Bzlmod dep in srcs (e.g. srcs = ["@score_baselibs//score/mw/log:log_stream_test.cpp"]), the clang-tidy aspect processes it and flags issues in that external file — even though the downstream project has no control over it (can't add // NOLINT or #pragma).

make_clang_tidy_aspect already applies angle_includes_are_system = True to suppress warnings from external headers. An analogous mechanism for source files from external repos would eliminate the need for per-target no-clang-tidy workarounds in downstream projects.

Tracking workaround: eclipse-score/logging#77 (score/mw/log/detail/common/BUILD — log_stream_test).

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne bei der Implementierung von make_clang_tidy_aspect und vergleiche deren Umgang mit externen Dateien mit angle_includes_are_system. Reproduziere das Problem anhand des externen Bzlmod-Quellbeispiels aus dem Bericht; die Aufgabe ist erledigt, wenn Warnungen aus solchen Quelldateien ohne Workarounds pro Target unterdrückt werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
build-system, tooling
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
68/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.