search

napalm-automation / napalm-ios

Apache License 2.0
31 stars 40 forks source link

Potential fix for #98 #99

Closed mirceaulinic closed 7 years ago

mirceaulinic commented 7 years ago
Note: this has not been tested as I don't have an IOS device - I've just wrote some lines hoping they will work :)
coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.5%) to 68.858% when pulling ad9623b147ae37556e06a18dd4ee6d264779c76d on FIX-98 into 527b8d48ce03c78764a6fae2a0fc0d788e17bf37 on develop.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.5%) to 68.858% when pulling ad9623b147ae37556e06a18dd4ee6d264779c76d on FIX-98 into 527b8d48ce03c78764a6fae2a0fc0d788e17bf37 on develop.

ktbyers commented 7 years ago

I am going to use a mechanism similar to this, but for various reasons had to make quite a few changes pertaining to the replace and merge operations, so I am going to close this PR and submit a separate PR with this support.