search

job / aggregate6

IPv4 and IPv6 prefix aggregation tool
BSD 2-Clause "Simplified" License
144 stars 20 forks source link

added truncate #4

Closed afpd closed 6 years ago

afpd commented 6 years ago

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

coveralls commented 6 years ago

Coverage Status

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

job commented 6 years ago

@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 commented 6 years ago

Coverage Status

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

afpd commented 6 years ago

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

job commented 6 years ago

Thanks!