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

Adding -x option which allows user to specify a threshold and check if atleast those many hosts are active #138

Merged
merged 6 commits into from
Oct 30, 2018

Conversation

deepak0004
Copy link
Contributor

@deepak0004 deepak0004 commented Oct 17, 2018

The feature as asked by #99

@coveralls
Copy link

coveralls commented Oct 17, 2018

Coverage Status

Coverage increased (+0.02%) to 79.733% when pulling 03d821e on deepak0004:develop into 180c6dd on schweikert:develop.

@deepak0004
Copy link
Contributor Author

deepak0004 commented Oct 22, 2018

Hi @schweikert,
Can you review this PR?

src/fping.c Outdated Show resolved Hide resolved
src/fping.c Outdated Show resolved Hide resolved
src/fping.c Outdated Show resolved Hide resolved
src/fping.c Outdated Show resolved Hide resolved
src/fping.c Outdated Show resolved Hide resolved
src/fping.c Outdated Show resolved Hide resolved
…achable hosts, and changing initialisation of min_reachable
@schweikert schweikert merged commit 80067fc into schweikert:develop Oct 30, 2018
@schweikert
Copy link
Owner

Thanks! I will revert the option to be called --reachable again instead of --min_reachable, because I think it looks nicer.

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