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)
- 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