pyangbind
pyangbind copied to clipboard
Potential bugfix for #232
This is the proposed fix for #232 although I am still rather concerned that we don't have any tests which expose this reported issue.
I also had to do some small updates for the latest openconfig model changes.
And I included a minor version bump because, with this release, the xml serialization will be released as well!
Codecov Report
Merging #233 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #233 +/- ##
=======================================
Coverage 71.37% 71.37%
=======================================
Files 7 7
Lines 1806 1806
Branches 484 484
=======================================
Hits 1289 1289
Misses 382 382
Partials 135 135
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 52e4693...b70c62e. Read the comment docs.
I tested using the branch tarkatronic:issue-232, it does fix the issue mentioned in issue-232. Thanks!
Hi,
Could you please rebase to a recent versions of pyangbind ?
Thanks.
Hi,
Could you please rebase to a recent versions of pyangbind ?
Thanks.
Hi @JoseIgnacioTamayo, I have not used pyangbind in many years now, and am therefore no longer involved in the development. If this is still found to be a useful bugfix, I would encourage you to take over this change. Thank you!
https://github.com/robshakir/pyangbind/issues/232 was closed.