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

disable SystemCertPool error on Windows #1046

Closed
wants to merge 1 commit into from
Closed

disable SystemCertPool error on Windows #1046

wants to merge 1 commit into from

Conversation

wongsyrone
Copy link
Contributor

since it's not implemented

golang/go#16736

Signed-off-by: Syrone Wong [email protected]

since it's not implemented

golang/go#16736

Signed-off-by: Syrone Wong <[email protected]>
@codecov
Copy link

codecov bot commented Apr 13, 2018

Codecov Report

Merging #1046 into master will increase coverage by 0.12%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1046      +/-   ##
==========================================
+ Coverage   75.03%   75.15%   +0.12%     
==========================================
  Files         225      225              
  Lines        9989     9990       +1     
==========================================
+ Hits         7495     7508      +13     
+ Misses       1780     1770      -10     
+ Partials      714      712       -2
Impacted Files Coverage Δ
transport/internet/tls/config.go 71% <0%> (-0.72%) ⬇️
common/log/logger.go 84.37% <0%> (-3.13%) ⬇️
transport/internet/udp/hub.go 77.63% <0%> (+1.31%) ⬆️
transport/internet/udp/dispatcher.go 89.09% <0%> (+25.45%) ⬆️

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 e4b2498...8d4f6d7. Read the comment docs.

@wongsyrone
Copy link
Contributor Author

I'm not familiar with the golang philosophy. If you care the issue reported by codebeat, feel free to refine the code.

@DarienRaymond
Copy link
Contributor

Prefer conditional build as ccafce3

@wongsyrone
Copy link
Contributor Author

It's more elegant.

@TanNang
Copy link

TanNang commented Aug 18, 2018

golang/go#26770

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

3 participants