Skip to content

Expose core tlog-witness - #549

Merged
AlCutter merged 3 commits into
transparency-dev:mainfrom
AlCutter:expose_witness
Jun 11, 2026
Merged

Expose core tlog-witness#549
AlCutter merged 3 commits into
transparency-dev:mainfrom
AlCutter:expose_witness

Conversation

@AlCutter

Copy link
Copy Markdown
Contributor

This PR exposes the core witness and HTTP support, which together represent the tlog-witness implementation, in a new /witness package.

Some unused/obsolete code was removed during the process.

@codecov-commenter

codecov-commenter commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 44.44444% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 23.02%. Comparing base (e288f04) to head (02f3d17).

Files with missing lines Patch % Lines
witness/http.go 53.33% 7 Missing ⚠️
omniwitness/omniwitness.go 40.00% 5 Missing and 1 partial ⚠️
internal/persistence/testonly/persistence.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #549      +/-   ##
==========================================
- Coverage   24.20%   23.02%   -1.19%     
==========================================
  Files          31       30       -1     
  Lines        2115     2063      -52     
==========================================
- Hits          512      475      -37     
+ Misses       1515     1499      -16     
- Partials       88       89       +1     

☔ View full report in Codecov by Harness.
📢 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.

@AlCutter
AlCutter merged commit 36c93e7 into transparency-dev:main Jun 11, 2026
13 checks passed
@AlCutter
AlCutter deleted the expose_witness branch June 11, 2026 10:53
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