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.

Sign up for GitHub

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

Jump to bottom

added truncate #4

Merged
merged 2 commits into from
Dec 1, 2017
Merged

added truncate #4

merged 2 commits into from
Dec 1, 2017

Conversation

afpd
Copy link
Contributor

@afpd afpd commented Dec 1, 2017

'Classic' aggregate tool for IPv4 has useful -t option. This patch implements it for aggregate6.

@coveralls
Copy link

coveralls commented Dec 1, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-1.9%) to 98.077% when pulling a5c56e3 on afpd:master into 2b5de94 on job:master.

@job
Copy link
Owner

job commented Dec 1, 2017

@afpd thank you for this contribution!

Would you mind adding a test to tests/test_regression.py to cover the -t feature?

If you need help with that let me know

@coveralls
Copy link

coveralls commented Dec 1, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 91dbeeb on afpd:master into 2b5de94 on job:master.

@afpd
Copy link
Contributor Author

afpd commented Dec 1, 2017

Ok. Added test for the the -t option, is it OK?

@job job merged commit ddda8c9 into job:master Dec 1, 2017
@job
Copy link
Owner

job commented Dec 1, 2017

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@afpd @coveralls @job