Closed Habush closed 2 years ago
Since I commented on the last pull, I feel obliged to comment here as well. I skimmed this, and it looks fine (after my very superficial skim.)
Thanks @linas. All the targets build without an issue. Some tests still fail which I will fix in another PR
What's the status of this PR? I want to send another PR that fixes the failing unit-tests that is based on this one. Or should include them it in this one?
What's the status of this PR?
According to the stats, the lead developer is @kasimebrahim guided by @ngeiswei so they would have to approve and merge. I see that @Bitseat and @Yidnekachew have contributed; maybe they have something to add?
I can merge (or tell you to merge) as a last resort, but these kinds of procedural questions need to be in the hands of the package maintainers.
Thanks @Habush !
This PR fixes #82 and uses the second approach mentioned in the issue