alpacahq / marketstore

DataFrame Server for Financial Timeseries Data
Apache License 2.0
1.89k stars 232 forks source link

feat(di): introduce Dependency Injection #606

Closed dakimura closed 2 years ago

dakimura commented 2 years ago

WHAT

codecov-commenter commented 2 years ago

Codecov Report

Merging #606 (4399161) into master (17c3cfe) will increase coverage by 0.03%. The diff coverage is 34.48%.

@@            Coverage Diff             @@
##           master     #606      +/-   ##
==========================================
+ Coverage   48.43%   48.46%   +0.03%     
==========================================
  Files         136      136              
  Lines       12966    12899      -67     
==========================================
- Hits         6280     6252      -28     
+ Misses       6067     6037      -30     
+ Partials      619      610       -9     
Impacted Files Coverage Δ
cmd/connect/session/local_api_client.go 0.00% <0.00%> (ø)
utils/config.go 1.66% <0.00%> (+0.01%) :arrow_up:
executor/wal.go 54.19% <33.33%> (-0.06%) :arrow_down:
executor/instance.go 100.00% <100.00%> (+52.56%) :arrow_up:
executor/written.go 96.66% <100.00%> (ø)
replication/grpc_server.go 91.42% <100.00%> (ø)