napalm-automation / napalm-junos

Apache License 2.0
22 stars 42 forks source link

Fixes #150 - Adds more actions to config detection. #151

Closed zachmoody closed 7 years ago

zachmoody commented 7 years ago
Seems this approach has a slight edge over using regex. I had a look at some of the unittests but could find a good fit for this, but admittedly I haven't used pytest before. If someone wanted to point me in the right direction to get started I could add some.
mirceaulinic commented 7 years ago

Merging - the build errors are not related to this PR, to be fixed by https://github.com/napalm-automation/napalm-junos/issues/149

mirceaulinic commented 7 years ago

Thanks @zachmoody