Skip to content

fix: better resource accounting - #158

Merged
rvagg merged 3 commits into
masterfrom
fix/resource-accounting
Oct 9, 2026
Merged

rvagg merged 3 commits into
masterfrom
fix/resource-accounting

Conversation

@LexLuthr

@LexLuthr LexLuthr commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@LexLuthr
LexLuthr requested a review from rvagg as a code owner October 9, 2026 10:59
@codecov-commenter

codecov-commenter commented Oct 9, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 81.15016% with 59 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.42%. Comparing base (b996f44) to head (fdef059).
⚠️ Report is 9 commits behind head on master.

Files with missing lines Patch % Lines
websocket.go 77.73% 54 Missing and 5 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
+ Coverage   74.50%   76.42%   +1.92%     
==========================================
  Files          13       13              
  Lines        1620     1820     +200     
==========================================
+ Hits         1207     1391     +184     
- Misses        314      332      +18     
+ Partials       99       97       -2     
Files with missing lines Coverage Δ
client.go 83.71% <100.00%> (+1.37%) ⬆️
handler.go 75.26% <100.00%> (+3.00%) ⬆️
options_server.go 88.46% <100.00%> (+19.71%) ⬆️
server.go 81.25% <100.00%> (+4.03%) ⬆️
websocket.go 79.09% <77.73%> (+0.03%) ⬆️

... and 2 files with indirect coverage changes

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

@LexLuthr
LexLuthr force-pushed the fix/resource-accounting branch from dd3a47a to 0a8dfda Compare October 9, 2026 11:18
@rvagg
rvagg merged commit 69856d1 into master Oct 9, 2026
7 checks passed
@rvagg
rvagg deleted the fix/resource-accounting branch October 9, 2026 11:38
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