Kidus Tegene
Kidus Tegene
Hey @nukeop, I am new here and would like to work on this issue if it is still available. As of right now, I am thinking to continue @thathva's approach...
Hey @nukeop I apologize for the hassle I created with the force push. Is there a way we can fix it? Please let me know if you need me to...
As far as the testing goes, merging works with me. I run some manual tests with small log sizes and the output seems to be as we want it. This...
Also, I was able to merge your branch with my master branch locally. Would you suggest that I push these changes to my GitHub fork associated with this PR?
Sounds good, I have pushed them
Sure, no problem. I have removed the last test, let me know if that works
Yea, no problem. How about now
I guess I learnt lesson to not force push for future, thanks or bearing with me. I can get started on copying to new branch