search

napalm-automation / napalm-junos

Apache License 2.0
22 stars 42 forks source link

Detect the format of the config and try loading it #132

Closed mirceaulinic closed 7 years ago

mirceaulinic commented 7 years ago
mirceaulinic commented 7 years ago

@dbarrosop @ktbyers I made some tests, this seems to work fine. Please let me know your suggestions / improvements etc.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-1.4%) to 83.883% when pulling a563339ff289c393d66acd8e5dccdfda60b71984 on mirceaulinic:cfg-fmt into 26c628b0ab7ed7d7636a0697ea6f0f626e428565 on napalm-automation:develop.

dbarrosop commented 7 years ago

Could you add tests? I think this is one of those methods that deserves it :) Other than that LGTM!