Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.5.17 #176

Merged
merged 6 commits into from
Nov 24, 2023
Merged

v0.5.17 #176

merged 6 commits into from
Nov 24, 2023

Conversation

oke11o
Copy link
Collaborator

@oke11o oke11o commented Nov 24, 2023

No description provided.

github badges
github actions release
remove badge
GetMapValue refactoring
$ go test -bench=. -count=5
goos: darwin
goarch: amd64
pkg: github.com/yandex/pandora/lib/mp
cpu: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
BenchmarkGetMapValue-12     	   45223	     24348 ns/op
BenchmarkGetMapValue-12     	   51969	     23631 ns/op
BenchmarkGetMapValue-12     	   52303	     24771 ns/op
BenchmarkGetMapValue-12     	   52562	     23441 ns/op
BenchmarkGetMapValue-12     	   49144	     23131 ns/op
BenchmarkGetMapValue2-12    	   49866	     23739 ns/op
BenchmarkGetMapValue2-12    	   47876	     23441 ns/op
BenchmarkGetMapValue2-12    	   49641	     24082 ns/op
BenchmarkGetMapValue2-12    	   50052	     23706 ns/op
BenchmarkGetMapValue2-12    	   51109	     23714 ns/op
PASS
ok  	github.com/yandex/pandora/lib/mp	15.501s
@codecov-commenter
Copy link

codecov-commenter commented Nov 24, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (8fee55f) 63.43% compared to head (bbfbd6e) 64.05%.
Report is 4 commits behind head on master.

Files Patch % Lines
core/coreutil/waiter.go 55.55% 4 Missing ⚠️
lib/mp/map.go 98.38% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
+ Coverage   63.43%   64.05%   +0.61%     
==========================================
  Files         107      107              
  Lines        5325     5338      +13     
==========================================
+ Hits         3378     3419      +41     
+ Misses       1722     1702      -20     
+ Partials      225      217       -8     
Flag Coverage Δ
go-1.20.x 63.95% <93.58%> (+0.57%) ⬆️
go-1.21.x 63.91% <93.58%> (+0.48%) ⬆️
macOS 64.03% <93.58%> (+0.59%) ⬆️
ubuntu 63.90% <93.58%> (+0.52%) ⬆️
unit 64.05% <93.58%> (+0.61%) ⬆️

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.

@oke11o oke11o merged commit dda167d into master Nov 24, 2023
4 of 5 checks passed
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.

2 participants