Closed mrshu closed 9 years ago
to @JakubNvk cc @pepol
@JakubNvk I do not want to be mean but I do not see how this could work before.
This only shows that we need tests and we need them ASAP.
@mrshu LGTM, but it's pretty sad to see a typo in commit message that fixes typos (thorw
), so please fix that.
@pepol thanks, this is precisely what review process is for.
In other news, this is the 100th PR/issue!
@mrshu LGTM. Also yay 100th PR!
@mrshu and so what? Unless it's not 128th, it's not a round number! :-D