Skip to content

ci: Fix clang-tidy workflow for dnsdist and recursor#17579

Open
rgacogne wants to merge 3 commits into
PowerDNS:masterfrom
rgacogne:fix-clang-tidy-dnsdist-rec
Open

ci: Fix clang-tidy workflow for dnsdist and recursor#17579
rgacogne wants to merge 3 commits into
PowerDNS:masterfrom
rgacogne:fix-clang-tidy-dnsdist-rec

Conversation

@rgacogne

Copy link
Copy Markdown
Member

Short description

git-filter was no longer looking into the correct compilation database, likely since we removed autotools support.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • read and accepted the Developer Certificate of Origin document, including the AI Policy, and added a "Signed-off-by" to my commits
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

rgacogne added 2 commits June 19, 2026 12:02
Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>
… builds

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>
@rgacogne rgacogne added this to the common-soon milestone Jun 19, 2026
@coveralls

coveralls commented Jun 19, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 27823940000

Coverage decreased (-0.003%) to 71.074%

Details

  • Coverage decreased (-0.003%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 7010 coverage regressions across 81 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

7010 previously-covered lines in 81 files lost coverage.

Top 10 Files by Coverage Loss Lines Losing Coverage Coverage
pdns/recursordist/pdns_recursor.cc 1099 46.19%
pdns/recursordist/lua-recursor4.cc 681 8.69%
pdns/recursordist/rec-main.cc 518 51.14%
pdns/recursordist/rec-tcp.cc 450 35.7%
pdns/recursordist/rec_channel_rec.cc 391 19.44%
pdns/recursordist/lwres.cc 350 37.85%
pdns/recursordist/rec-lua-conf.cc 347 19.57%
pdns/recursordist/rec-rust-lib/cxxsupport.cc 341 57.88%
pdns/recursordist/syncres.cc 339 77.13%
pdns/recursordist/rpzloader.cc 291 43.45%

Coverage Stats

Coverage Status
Relevant Lines: 171239
Covered Lines: 133399
Line Coverage: 77.9%
Relevant Branches: 81838
Covered Branches: 46473
Branch Coverage: 56.79%
Branches in Coverage %: Yes
Coverage Strength: 5537811.53 hits per line

💛 - Coveralls

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants