Skip to content

Lint all headers and add some tests#474

Merged
Anilm3 merged 1 commit into
masterfrom
anilm3/lint-all-headers
Oct 28, 2025
Merged

Lint all headers and add some tests#474
Anilm3 merged 1 commit into
masterfrom
anilm3/lint-all-headers

Conversation

@Anilm3

@Anilm3 Anilm3 commented Oct 28, 2025

Copy link
Copy Markdown
Collaborator

As per the title, some clerical improvements.

@Anilm3
Anilm3 requested a review from a team as a code owner October 28, 2025 14:45
chatgpt-codex-connector[bot]

This comment was marked as resolved.

@Anilm3
Anilm3 force-pushed the anilm3/lint-all-headers branch from c316c59 to 4865ac6 Compare October 28, 2025 14:51
@Anilm3
Anilm3 temporarily deployed to dd-protected-coverage October 28, 2025 15:00 — with GitHub Actions Inactive
@codecov-commenter

codecov-commenter commented Oct 28, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.19%. Comparing base (10c6264) to head (3038199).
⚠️ Report is 16 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #474   +/-   ##
=======================================
  Coverage   85.19%   85.19%           
=======================================
  Files         189      189           
  Lines        9225     9225           
  Branches     4178     4178           
=======================================
  Hits         7859     7859           
  Misses        531      531           
  Partials      835      835           
Flag Coverage Δ
waf_test 85.19% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@datadog-datadog-prod-us1

This comment has been minimized.

@github-actions

github-actions Bot commented Oct 28, 2025

Copy link
Copy Markdown

Artifact Size Comparison 📦

Artifact Previous Release This PR Difference
darwin-arm64::libddwaf.a 90006808 93338872 0.03%
darwin-arm64::libddwaf.a.stripped 4579664 4920392 0.07%
darwin-arm64::libddwaf.dylib 1963024 2048736 0.04%
darwin-universal::libddwaf.a 181421048 188348408 0.03%
darwin-universal::libddwaf.a.stripped 9765560 10492280 0.07%
darwin-universal::libddwaf.dylib 4142096 4309728 0.04%
darwin-x86_64::libddwaf.a 91414192 95009488 0.03%
darwin-x86_64::libddwaf.a.stripped 5185848 5571840 0.07%
darwin-x86_64::libddwaf.dylib 2150896 2237824 0.04%
linux-aarch64::libddwaf.a 73159398 76100842 0.04%
linux-aarch64::libddwaf.a.stripped 11852314 12387414 0.04%
linux-aarch64::libddwaf.so 2463264 2513896 0.02%
linux-armv7::libddwaf.a 64719602 67256200 0.03%
linux-armv7::libddwaf.a.stripped 10852830 11374276 0.04%
linux-armv7::libddwaf.so 2148796 2171356 0.01%
linux-i386::libddwaf.a 62843294 65326128 0.03%
linux-i386::libddwaf.a.stripped 9380114 9834100 0.04%
linux-i386::libddwaf.so 2392820 2435616 0.01%
linux-x86_64::libddwaf.a 73646022 76603200 0.04%
linux-x86_64::libddwaf.a.stripped 11671354 12181236 0.04%
linux-x86_64::libddwaf.so 2660096 2711680 0.01%
windows-arm64::ddwaf.dll 4788736 4940800 0.03%
windows-arm64::ddwaf.lib 11698 16410 0.40%
windows-arm64::ddwaf_static.lib 57913956 60474674 0.04%
windows-win32::ddwaf.dll 3368448 3478528 0.03%
windows-win32::ddwaf.lib 11922 16726 0.40%
windows-win32::ddwaf_static.lib 49534552 51791730 0.04%
windows-x64::ddwaf.dll 4101632 4207616 0.02%
windows-x64::ddwaf.lib 11698 16410 0.40%
windows-x64::ddwaf_static.lib 57410292 59908952 0.04%

@pr-commenter

pr-commenter Bot commented Oct 28, 2025

Copy link
Copy Markdown

Benchmarks clang-pgo

Benchmark execution time: 2025-10-28 15:28:34

Comparing candidate commit 4865ac6 in PR branch anilm3/lint-all-headers with baseline commit 10c6264 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1 metrics, 0 unstable metrics.

@Anilm3
Anilm3 force-pushed the anilm3/lint-all-headers branch from 4865ac6 to 3038199 Compare October 28, 2025 16:00
@Anilm3
Anilm3 enabled auto-merge (squash) October 28, 2025 16:05
@Anilm3
Anilm3 temporarily deployed to dd-protected-coverage October 28, 2025 16:11 — with GitHub Actions Inactive
@Anilm3
Anilm3 merged commit f6d1998 into master Oct 28, 2025
53 of 54 checks passed
@Anilm3
Anilm3 deleted the anilm3/lint-all-headers branch October 28, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants