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

Add partial (client) binding support for BSD #260

Merged
merged 3 commits into from
Mar 13, 2019
Merged

Add partial (client) binding support for BSD #260

merged 3 commits into from
Mar 13, 2019

Conversation

dmgk
Copy link
Contributor

@dmgk dmgk commented Mar 13, 2019

Related to #240

@codecov
Copy link

codecov bot commented Mar 13, 2019

Codecov Report

Merging #260 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #260   +/-   ##
=======================================
  Coverage   75.31%   75.31%           
=======================================
  Files          68       68           
  Lines        3083     3083           
=======================================
  Hits         2322     2322           
  Misses        665      665           
  Partials       96       96

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 4795153...37bcb0f. Read the comment docs.

@pmazzini
Copy link
Collaborator

@dmgk Thanks for your contribution! Doesn't it need a build constraint / build tag?

@dmgk
Copy link
Contributor Author

dmgk commented Mar 13, 2019

@pmazzini It does have one, currently constraining the build to freebsd, openbsd and netbsd only: https://github.com/insomniacslk/dhcp/pull/260/files#diff-e78aed4e493232b381170e9512c08c72R1

Copy link
Collaborator

@pmazzini pmazzini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I am not sure what I was looking at. LGTM

@pmazzini
Copy link
Collaborator

@dmgk Would you mind closing and reopening the PR (it is just two clicks) to trigger a new travis build?

@dmgk dmgk closed this Mar 13, 2019
@dmgk dmgk reopened this Mar 13, 2019
@pmazzini
Copy link
Collaborator

@dmgk Thanks for your contribution!

@pmazzini pmazzini merged commit b57e89f into insomniacslk:master Mar 13, 2019
@dmgk
Copy link
Contributor Author

dmgk commented Mar 13, 2019

@pmazzini Thanks for merging!

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.

3 participants