在abandoned的PR上comment？

新手刚来这个组，和一个印度小哥合作临时做个项目
他私下联系我让我帮忙看个PR，我一看是被他手动abandoned的。。一眼就看出我需要comment很多，他说我先comment，他修改后再提交一个正式PR
我个人认为我在一个abandoned PR上评论，最后根本不知道谁comment、谁contribute了，毕竟最后是废弃的。不知道是不是理解错了？感谢前辈指导

你就告诉他，等他新的diff出来之后再comment，因为你需要看最新的diff才能知道需要评论哪些。

can you send a new pr, for better tracking and broader visibility.

可以理解吧。特别是在toxic的公司里的话，如果某个DIFF有很多comment/revision的话，performance review的时候会被拿出来挑刺。
如果你想维持好的私交的话，可以这么做。如果只想维持一般同事关系的话，就让他发正式PR。

如果我对这个code change特别没有自信的话我会发draft给人review，也会review别人的draft。

請他直接發PR，這樣比較好追蹤所有的更動

这不要吧，不等于你帮他检查作业了吗，除非他是给你干活，那就不讲究了，不然说等有一个ready for review的pr会review

可以理解pr不自信的时候，先让别人在draft pr上写评论，改得差不多了，再publish让更多人写评论。这样既可以减少review工作量，也可以让所有comment保留记录。

补充内容 (2023-01-18 02:41 +8:00):
但是让在abandoned pr上评论也就算了。虽然review带来的credit几乎没有，但也没必要在abandoned pr上review

补充内容 (2023-01-18 02:44 +8:00):
我觉得可能是manager提醒他的pr上comment有点多。
我刚入职那会manager在1:1上跟我这么说过，我没当回事。现在想来，应该是manager从reviewer那边拿到的feedback，侧面提醒我。因为manager不碰code，不太可能会去点开我的pr看

哈哈这个小哥心机很多啊

哈哈烙印这种小操作挺多的 学会理直气壮的拒绝 之后会轻松很多

关键是你无法阻止对方abandon pr，即使最新的pr你在comment之后，对方依然可以abandon，然后发一个更新修改过的。