Skip to content

Update chunk-reconfig to RFC 6525#432

Open
philipch07 wants to merge 1 commit into
mainfrom
pch07/update-chunk-reconfig
Open

Update chunk-reconfig to RFC 6525#432
philipch07 wants to merge 1 commit into
mainfrom
pch07/update-chunk-reconfig

Conversation

@philipch07

@philipch07 philipch07 commented Nov 8, 2025

Copy link
Copy Markdown
Member

Description

updates and completes TODOs in chunk_reconfig.go according to RFC 6525.

Reference issue

n/a

@philipch07 philipch07 force-pushed the pch07/update-chunk-reconfig branch from 5d892f9 to d4bc0d0 Compare November 8, 2025 21:29
@philipch07 philipch07 marked this pull request as ready for review November 8, 2025 21:29
@codecov

codecov Bot commented Nov 8, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 60.86957% with 27 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.52%. Comparing base (9ee8305) to head (d4bc0d0).

Files with missing lines Patch % Lines
chunk_reconfig.go 64.91% 16 Missing and 4 partials ⚠️
association.go 41.66% 4 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #432      +/-   ##
==========================================
- Coverage   83.90%   83.52%   -0.38%     
==========================================
  Files          51       51              
  Lines        3448     3484      +36     
==========================================
+ Hits         2893     2910      +17     
- Misses        416      432      +16     
- Partials      139      142       +3     
Flag Coverage Δ
go 83.52% <60.86%> (-0.38%) ⬇️

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.

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.

1 participant