Skip to content

chore: address TODO comments - #576

Open
azkrishpy wants to merge 5 commits into
mainfrom
todos
Open

azkrishpy wants to merge 5 commits into
mainfrom
todos

Conversation

@azkrishpy

@azkrishpy azkrishpy commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Issue #, if available: N/A

Description of changes:

Base branch for the TODO-cleanup chore stack. Each stacked PR merges here; this merges to main once as a single change.

Carries only test-side housekeeping: drops an unused mock-server field and its trailing whitespace, and raises the connection-monitor test interval from 1 to 2 seconds because the Java binding requires at least 2, which also matches the value the adjacent validity test already uses.

No library code touched.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov-commenter

codecov-commenter commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.02%. Comparing base (2b563f8) to head (b32d9ee).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #576      +/-   ##
==========================================
+ Coverage   79.84%   80.02%   +0.18%     
==========================================
  Files          28       28              
  Lines       12105    12105              
==========================================
+ Hits         9665     9687      +22     
+ Misses       2440     2418      -22     

☔ 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.

@github-actions github-actions Bot added the patch ABI backward-compatible (patch release) label Sep 8, 2026
@azkrishpy
azkrishpy marked this pull request as ready for review September 9, 2026 22:26
@azkrishpy
azkrishpy requested a review from a team as a code owner September 9, 2026 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch ABI backward-compatible (patch release)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants