You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have a general comment, "who is the receiver" comment, it has two points:
A good practice is to put all the function of a receiver in one file.
the receiver is not just another parameter, the main context of the function should be the receiver itself
if you still think it is better not to put a func in another file, at list put all the functions of the receiver together.
in this function, I understand why you do not want to put it in the function of detailedConn, the context of this function is explainability, not detailedConn. I dont think that detailedConn should be the receiver here, just a parameter.
I have a general comment, "who is the receiver" comment, it has two points:
in this function, I understand why you do not want to put it in the function of detailedConn, the context of this function is explainability, not detailedConn. I dont think that detailedConn should be the receiver here, just a parameter.
Originally posted by @haim-kermany in #627 (comment)
The text was updated successfully, but these errors were encountered: