Skip to content
New issue

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

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

Already on GitHub? # to your account

more major fixes for #7 and more tests #10

Merged
merged 2 commits into from
Jul 6, 2021

Conversation

gebv
Copy link
Owner

@gebv gebv commented Jul 6, 2021

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jul 6, 2021

Codecov Report

Merging #10 (80e8360) into master (c2741d7) will increase coverage by 0.17%.
The diff coverage is 78.26%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
+ Coverage   80.40%   80.57%   +0.17%     
==========================================
  Files           8        8              
  Lines         398      417      +19     
==========================================
+ Hits          320      336      +16     
- Misses         64       66       +2     
- Partials       14       15       +1     
Flag Coverage Δ
unittests 80.57% <78.26%> (+0.17%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
store.go 90.09% <78.26%> (-3.39%) ⬇️
rand.go 100.00% <0.00%> (+22.22%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c2741d7...80e8360. Read the comment docs.

@gebv gebv merged commit 34baa8c into master Jul 6, 2021
@gebv gebv deleted the bug/ambiguity_between_const_and_param branch July 6, 2021 17:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants