NXOS protocol param to standarized 'transport' param

napalm-automation / napalm-nxos

Apache License 2.0

9 stars 21 forks source link

NXOS protocol param to standarized 'transport' param #94

Closed The-Loeki closed 7 years ago

The-Loeki commented 7 years ago

Switch the optional argument nxos_protocol to the new, standarized transport

References:

https://github.com/napalm-automation/napalm-ios/pull/159
https://github.com/napalm-automation/napalm-ios/issues/132
https://github.com/napalm-automation/napalm-eos/pull/167

The-Loeki commented 7 years ago

@dbarrosop @mirceaulinic I've taken the liberty to update this driver as well if that's ok with you guys

coveralls commented 7 years ago

Coverage remained the same at 68.831% when pulling 1ab5bbfb2ecbb3991bba412d7b458f554fdb623d on The-Loeki:develop into fee62b79cb83b52305babbe1bfe8e5aac7a47097 on napalm-automation:develop.

dbarrosop commented 7 years ago

Great!

coveralls commented 7 years ago

Coverage decreased (-0.2%) to 68.669% when pulling b1764f8aec687dbcf25976f9c37fc783a85d0486 on The-Loeki:develop into fee62b79cb83b52305babbe1bfe8e5aac7a47097 on napalm-automation:develop.