search

alpacahq / marketstore

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

fix: lint issues for the command line interface #594

Closed dakimura closed 2 years ago

dakimura commented 2 years ago

WHAT

codecov-commenter commented 2 years ago

Codecov Report

Merging #594 (6018128) into master (c7d1c72) will increase coverage by 0.07%. The diff coverage is 18.71%.

@@            Coverage Diff             @@
##           master     #594      +/-   ##
==========================================
+ Coverage   48.75%   48.83%   +0.07%     
==========================================
  Files         138      138              
  Lines       12946    12905      -41     
==========================================
- Hits         6312     6302      -10     
+ Misses       6072     6037      -35     
- Partials      562      566       +4
Impacted Files Coverage Δ
cmd/connect/session/help.go 0.00% <0.00%> (ø)
cmd/connect/session/load.go 0.00% <0.00%> (ø)
cmd/connect/session/show.go 0.00% <0.00%> (ø)
cmd/connect/session/sql.go 0.00% <0.00%> (ø)
cmd/connect/session/trim.go 0.00% <0.00%> (ø)
cmd/connect/session/client.go 39.03% <20.95%> (-0.43%) :arrow_down:
cmd/connect/session/create.go 59.09% <43.75%> (-1.10%) :arrow_down:
contrib/ondiskagg/aggtrigger/aggtrigger.go 54.76% <100.00%> (ø)
utils/rpc/msgpack2/client.go 67.74% <100.00%> (+4.77%) :arrow_up:
frontend/stream/stream.go 58.47% <0.00%> (-12.72%) :arrow_down:
sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication