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

refactor(sync)!: use options to init metrics #121

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Conversation

Wondertan
Copy link
Member

Self-explanatory

@codecov-commenter
Copy link

Codecov Report

Merging #121 (e30550a) into main (26daa91) will decrease coverage by 0.14%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   69.64%   69.50%   -0.14%     
==========================================
  Files          37       37              
  Lines        2958     2971      +13     
==========================================
+ Hits         2060     2065       +5     
- Misses        751      759       +8     
  Partials      147      147              
Files Coverage Δ
sync/sync.go 72.37% <40.00%> (-0.92%) ⬇️
sync/options.go 75.86% <0.00%> (-12.14%) ⬇️
sync/metrics.go 16.66% <18.18%> (+11.66%) ⬆️

@renaynay renaynay merged commit 27e85f9 into main Oct 12, 2023
2 checks passed
@renaynay renaynay deleted the sync/option-for-metrics branch October 12, 2023 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants