Closed mojtaba-esk closed 1 month ago
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
It's probably cleaner to pass the
*knuu.Knuu
struct to the testnet constructor but this is also fine
Thanks for the feedback. yes that could make it better. Most parts of the e2e could be refactored as for example I see a number of funcs have too many arguments which we could use options instead.
While attempting to use testnet pkg in the node repo. There is a need to use
knuu
object that is initialized by thetestnet.New()
to create new instances and interact withknuu
. Since it is not exported, this PR proposed to have a getter func for it, so users can interact with knuu easily.