Skip to content

chore: migrate HawkEye to v7 - #3952

Open
tisonkun wants to merge 3 commits into
masterfrom
codex/migrate-hawkeye-v7
Open

chore: migrate HawkEye to v7#3952
tisonkun wants to merge 3 commits into
masterfrom
codex/migrate-hawkeye-v7

Conversation

@tisonkun

@tisonkun tisonkun commented Aug 22, 2026

Copy link
Copy Markdown
Member

Summary

  • migrate the HawkEye configuration and CI installation to v7/latest
  • adapt the custom license-header script to v7's JSON report and exit-code model
  • remove the v6 version file and update contributor installation instructions
  • normalize existing headers to v7's canonical styles

Design Notes

The larger file count is header-only normalization: application behavior is unchanged. The migrated integration was validated in Alpine with Bash 5, jq, Git, and the Linux HawkEye v7 binary. The custom check reports success for 3,131 selected files with 0 changes, 0 conflicts, and 0 unsupported files. The generated PHP stub is excluded because cargo-php owns its exact header spacing; its checked-in output remains unchanged. Changed files also pass the repository trailing-whitespace check, and the web sources pass Prettier.

@github-actions github-actions Bot added the S-waiting-on-review PR is waiting on a reviewer label Aug 22, 2026
@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.77%. Comparing base (cc269ef) to head (94b8349).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3952      +/-   ##
============================================
- Coverage     83.85%   83.77%   -0.08%     
  Complexity     1358     1358              
============================================
  Files          1212     1212              
  Lines        166843   166739     -104     
  Branches     134304   134432     +128     
============================================
- Hits         139905   139688     -217     
+ Misses        23298    23266      -32     
- Partials       3640     3785     +145     
Components Coverage Δ
Rust Core 84.64% <ø> (+0.01%) ⬆️
Java SDK 66.67% <ø> (ø)
C# SDK 74.91% <ø> (-1.62%) ⬇️
Python SDK 90.13% <ø> (ø)
PHP SDK 84.48% <ø> (ø)
Node SDK 95.81% <ø> (-0.04%) ⬇️
Go SDK 68.32% <ø> (-0.05%) ⬇️
Files with missing lines Coverage Δ
foreign/csharp/Iggy_SDK/Contracts/StatsResponse.cs 100.00% <ø> (ø)
...oreign/csharp/Iggy_SDK/Contracts/StreamResponse.cs 100.00% <ø> (ø)
foreign/csharp/Iggy_SDK/Contracts/TopicResponse.cs 100.00% <ø> (ø)
foreign/csharp/Iggy_SDK/Enums/IdKind.cs 70.58% <ø> (-5.89%) ⬇️
foreign/node/src/client/client.connection.ts 96.46% <ø> (-0.01%) ⬇️
foreign/node/src/client/client.debug.ts 100.00% <ø> (ø)
foreign/node/src/client/client.socket.ts 95.68% <ø> (-0.01%) ⬇️
foreign/node/src/client/client.ts 97.19% <ø> (-0.02%) ⬇️
foreign/node/src/client/client.utils.ts 100.00% <ø> (ø)
foreign/node/src/stream/consumer-stream.ts 76.05% <ø> (-0.17%) ⬇️
... and 100 more

... and 67 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review PR is waiting on a reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant