Skip to content

Perf/adapter options - #221

Merged
fredbi merged 2 commits into
go-openapi:masterfrom
fredbi:perf/adapter-options
Jul 19, 2026
Merged

Perf/adapter options#221
fredbi merged 2 commits into
go-openapi:masterfrom
fredbi:perf/adapter-options

Conversation

@fredbi

@fredbi fredbi commented Jul 19, 2026

Copy link
Copy Markdown
Member

Change type

Please select: 🆕 New feature or enhancement|🔧 Bug fix'|📃 Documentation update

Short description

Fixes

Full description

Checklist

  • I have signed all my commits with my name and email (see DCO. This does not require a PGP-signed commit
  • I have rebased and squashed my work, so only one commit remains
  • I have added tests to cover my changes.
  • I have properly enriched go doc comments in code.
  • I have properly documented any breaking change.

fredbi added 2 commits July 19, 2026 18:49
* reuse the common pools utility
* option pattern is now zero-alloc

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@codecov

codecov Bot commented Jul 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.84211% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.29%. Comparing base (efb6f4e) to head (a130e07).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
jsonutils/adapters/easyjson/json/options.go 42.85% 8 Missing ⚠️
jsonutils/adapters/easyjson/json/pool.go 92.30% 1 Missing ⚠️
jsonutils/adapters/stdlib/json/pool.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
+ Coverage   92.20%   92.29%   +0.08%     
==========================================
  Files          57       57              
  Lines        2746     2674      -72     
==========================================
- Hits         2532     2468      -64     
+ Misses        170      164       -6     
+ Partials       44       42       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@fredbi
fredbi merged commit 50ff260 into go-openapi:master Jul 19, 2026
21 checks passed
@fredbi
fredbi deleted the perf/adapter-options branch July 19, 2026 20:32
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