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

update geoip to 20180925 #1289

Closed
wants to merge 1 commit into from
Closed

Conversation

wongsyrone
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Sep 30, 2018

Codecov Report

Merging #1289 into master will increase coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1289      +/-   ##
==========================================
+ Coverage   74.25%   74.38%   +0.12%     
==========================================
  Files         254      254              
  Lines       11149    11149              
==========================================
+ Hits         8279     8293      +14     
+ Misses       2073     2059      -14     
  Partials      797      797
Impacted Files Coverage Δ
common/log/logger.go 84.37% <0%> (-3.13%) ⬇️
app/proxyman/mux/mux.go 51.74% <0%> (-1.4%) ⬇️
proxy/shadowsocks/client.go 69.69% <0%> (+1.01%) ⬆️
proxy/vmess/inbound/inbound.go 77.48% <0%> (+1.57%) ⬆️
transport/internet/headers/http/http.go 81.56% <0%> (+2.12%) ⬆️
transport/internet/websocket/connection.go 73.58% <0%> (+7.54%) ⬆️
app/proxyman/mux/session.go 83.56% <0%> (+12.32%) ⬆️

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 865d4ff...1169b92. Read the comment docs.

@VictoriaRaymond
Copy link
Member

Please don't do this so frequently. GeoIP is only updated every 3 months or more, unless you have a strong reason.

@wongsyrone wongsyrone deleted the geoip-update branch September 30, 2018 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants