napalm-automation / napalm-junos

Apache License 2.0
22 stars 42 forks source link

Version 0.5.2 #99

Closed mirceaulinic closed 7 years ago

mirceaulinic commented 7 years ago
coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 86.207% when pulling b0ffdcd75f47766086b46c299c20d9613bbc30f5 on rel-0.5.2 into c40cc819d8de6695468869ac45a64a18045930ce on develop.

ktbyers commented 7 years ago

Did anyone run the unit tests against a real device? I think we should do this, given all the Python3 changes.

I can see if I can get this done, but it probably won't be until Friday.

dbarrosop commented 7 years ago

I did and seemed to work fine, great job : )

mirceaulinic commented 7 years ago

Yes, I have also tested it yesterday - forgot to mention.