Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 wong.syrone@gmail.com

since it's not implemented

golang/go#16736

Signed-off-by: Syrone Wong <wong.syrone@gmail.com>
@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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants