uniqush / uniqush-push

Uniqush is a free and open source software system which provides a unified push service for server side notification to apps on mobile devices.
http://uniqush.org
Apache License 2.0
1.53k stars 201 forks source link

Fix more golint notices, add golint to the CI checks #231

Closed TysonAndre closed 5 years ago

TysonAndre commented 5 years ago

And slightly change error messages emitted by uniqush-push in logs and HTTP responses

codecov-io commented 5 years ago

Codecov Report

Merging #231 into master will increase coverage by 0.05%. The diff coverage is 6.97%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #231      +/-   ##
==========================================
+ Coverage   24.67%   24.72%   +0.05%     
==========================================
  Files          30       30              
  Lines        2906     2900       -6     
==========================================
  Hits          717      717              
+ Misses       2071     2067       -4     
+ Partials      118      116       -2
Impacted Files Coverage Δ
restapi_simpleresponse.go 0% <ø> (ø) :arrow_up:
srv/apns/http_api/processor.go 62.23% <ø> (+1.39%) :arrow_up:
push/pushserviceconfig.go 0% <ø> (ø) :arrow_up:
srv/apns/binary_api/conn_manager.go 33.75% <ø> (ø) :arrow_up:
srv/apns/binary_api/processor.go 73.29% <ø> (-1.14%) :arrow_down:
push/pushservicetype.go 0% <ø> (ø) :arrow_up:
db/pushredisdb.go 12.82% <ø> (ø) :arrow_up:
push/errors.go 0% <ø> (ø) :arrow_up:
srv/apns/binary_api/pool.go 58.44% <ø> (ø) :arrow_up:
push/pushservicemngr.go 33.6% <ø> (ø) :arrow_up:
... and 8 more

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 f17e626...0177fac. Read the comment docs.