{"id":201365,"date":"2020-11-25T00:40:18","date_gmt":"2020-11-24T16:40:18","guid":{"rendered":"http:\/\/4563.org\/?p=201365"},"modified":"2020-11-25T00:40:18","modified_gmt":"2020-11-24T16:40:18","slug":"%e6%88%91%e5%8f%91%e7%8e%b0-%e6%88%91%e5%85%ac%e5%8f%b8%e7%9a%84-review-code-%e8%bf%99%e4%b8%aa%e7%9b%ae%e7%9a%84%e5%b7%b2%e7%bb%8f%e5%8f%98%e8%b4%a8%e4%ba%86%e3%80%82","status":"publish","type":"post","link":"http:\/\/4563.org\/?p=201365","title":{"rendered":"\u6211\u53d1\u73b0 \u6211\u516c\u53f8\u7684 review code \u8fd9\u4e2a\u76ee\u7684\u5df2\u7ecf\u53d8\u8d28\u4e86\u3002"},"content":{"rendered":"<div>\n<div>\n<div>\n<h1>                  \u6211\u53d1\u73b0 \u6211\u516c\u53f8\u7684 review code \u8fd9\u4e2a\u76ee\u7684\u5df2\u7ecf\u53d8\u8d28\u4e86\u3002               <\/h1>\n<p> <\/p>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : kpppp <\/span>  <span><i><\/i> 5<\/span> <\/div>\n<div> <\/div>\n<\/p><\/div>\n<\/p><\/div>\n<\/p><\/div>\n<div isfirst=\"1\"> <\/p>\n<p>\u6211\u5446\u8fc7\u51e0\u5bb6\u516c\u53f8\uff0c\u90fd\u662f\u5343\u4eba\u4ee5\u4e0a\u7684\u89c4\u6a21\u3002\u4f46\u662f review code \u7ed9\u6211\u7684\u7edf\u4e00\u611f\u89c9\u5c31\u662f [\u5b98\u50da] \u3002<\/p>\n<p>\u660e\u663e\u7684\u8bed\u6cd5\u9519\u8bef\u9879\u76ee\u7ec4 leader \u4e5f\u770b\u4e0d\u51fa\u6765\uff0c\u53ea\u662f\u770b\u4e00\u773c\uff0c\u8fd9\u4e9b\u6709+2 \u6743\u9650\u7684\u4eba\uff0c\u5927\u591a\u6570\u90fd\u6ca1\u6709\u505a\u8fc7\u4f60\u7684\u9879\u76ee\uff0c\u53ea\u662f\u7ea7\u522b\u9ad8\u3002\u5728\u5165\u5e93\u8fd9\u4e00\u5173\uff0c\u5361\u6b7b\u4f60\uff0c\u6bcf\u6b21\u90fd\u8981\u53d1\u6d88\u606f\uff0c\u544a\u77e5\u5bf9\u65b9\uff0c\u8bf7\u5e2e\u6211 review \uff01<\/p>\n<p>\u6bd4\u5982\u6211\u4eec\u6700\u8fd1\u7684\u4e00\u4e2a\u653f\u7b56\uff1a \u9886\u5bfc\u8bf4:\u4e3a\u4e86\u63d0\u9ad8\u6211\u4eec\u4ee3\u7801\u8d28\u91cf\uff0c\u6240\u4ee5\u6211\u4eec\u8981\u4e00\u8d77 review \u3002\u5c31\u662f\u8ba9\u5199 c \u6216 c++\u7684\u4eba\u548c\u5199 java \u6216 kotlin \u7684\u4eba\u4e92\u76f8 review code \u3002<br \/> \u5b9e\u884c\u4e86 2 \u5929\u4e86\uff0c\u666e\u901a\u5de5\u7a0b\u5e08\u8bf4\u7684\u6700\u591a\u7684\u51e0\u53e5\u8bdd:<br \/> 1.\u770b\u4e0d\u61c2\uff0c\u4e0d\u77e5\u9053\u4f60\u5199\u7684\u5565\u3002<br \/> 2.\u6548\u7387\u597d\u4f4e\u554a\uff0c\u6ca1\u5565\u9e1f\u7528\u3002<\/p>\n<p>\u9ad8\u7ea7\u4e0e\u4e13\u5bb6\u5de5\u7a0b\u5e08\u8bf4\u7684\u6700\u591a\u7684\u51e0\u53e5\u8bdd:<br \/> 1.\u4f60\u8fd9\u4e2a\u4ee3\u7801\u6709\u70b9\u5197\u4f59\u554a<br \/> 2.\u8fd9\u4e2a\u6ca1\u6709\u95ee\u9898\u5427<br \/> 3.ok \uff0c\u6211\u9a6c\u4e0a+1<\/p>\n<p>\u9996\u5e2d\u4e0e\u7ecf\u7406\u8bf4\u7684\u8bdd\uff1a<br \/> 1.ok<br \/> 2.\u5df2\u7ecf+2<\/p>\n<p>\u4e8e\u662f\u6211\u76f4\u63a5\u628a\u6574\u4e2a\u9879\u76ee\u7ec4\u7684\u4eba\u90fd\u52a0\u5165\u4e86 review\uff0c\u7136\u540e\u5f15\u8d77\u4e86 leader \u7684\u4e0d\u6ee1,\u8bf4\u6211\u592a\u81ea\u6211\u4e86.\u4e8e\u662f\u6211\u53c8\u4e00\u4e2a\u4e00\u4e2a\u7684 remove reviewer \u3002<\/p>\n<p>\u54ce\uff0c\u96be\u554a\u3002<\/p>\n<\/p><\/div>\n<div> <b>\u5927\u4f6c\u6709\u8a71\u8aaa<\/b> (<span>54<\/span>)        <\/div>\n<div> <\/div>\n<\/p><\/div>\n<\/p><\/div>\n<ul>\n<li data-pid=\"4195209\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : wuzhouhui <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u6211\u4eec\u7684\u95ee\u9898\u7684\u79ef\u6781\u6027\u4e0d\u9ad8, \u56e0\u4e3a\u8bc4\u5ba1\u4ee3\u7801\u4f1a\u803d\u8bef\u81ea\u5df1\u5199\u4ee3\u7801\u7684\u65f6\u95f4, \u81ea\u5df1\u4ee3\u7801\u5199\u4e0d\u5b8c, \u9886\u5bfc\u5c31\u4f1a\u8d23\u602a. \u800c\u5728\u5b89\u6392\u5f00\u53d1\u65f6\u95f4\u7684\u65f6\u5019, \u662f\u4e0d\u4f1a\u8003\u8651\u5230\u8bc4\u5ba1\u7684\u8017\u65f6. \u53e6\u5916, \u6709\u4e9b\u4eba\u6c34\u5e73\u592a\u5dee, \u6539\u4e86\u51e0\u6b21\u90fd\u6709\u95ee\u9898, \u7136\u540e\u5c31\u4e0d\u60f3\u770b\u4e86. \u53e6\u5916\u5c31\u662f, \u516c\u53f8\u5b9a\u7684\u8bc4\u5ba1\u5de5\u5177\u6548\u7387\u5b9e\u5728\u592a\u4f4e\u4e86. \u6240\u4ee5\u73b0\u5728, \u901a\u5e38\u662f\u9886\u5bfc\u548c\u5bf9\u76f8\u5173\u4ee3\u7801\u6bd4\u8f83\u719f\u6089\u7684\u5458\u5de5\u4f1a\u8d1f\u8d23\u8bc4\u5ba1\u4ee3\u7801. \u800c\u6c34\u5e73\u5dee\u4e00\u70b9\u7684, \u6211\u4e5f\u4e0d\u60f3\u8ba9\u4ed6\u4eec\u770b, \u4ed6\u4eec\u63d0\u7684\u610f\u89c1\u5b9e\u5728\u6ca1\u4ef7\u503c.                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195210\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : wuzhouhui <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8fd8\u6709\u4e00\u4e2a, \u90a3\u5c31\u662f\u516c\u53f8\u7684\u7f16\u7801\u98ce\u683c\u592a\u8ba8\u538c\u4e86, \u975e\u5e38\u7684\u5570\u55e6, \u5197\u4f59, git \u65e5\u5fd7\u53ea\u5199\u4e00\u53e5\u8bdd\u751a\u81f3\u4e0d\u5199. \u6211\u6bd4\u8f83\u559c\u6b22\u5185\u6838\u90a3\u6837\u7b80\u6d01\u6e05\u6670\u7684\u7f16\u7801\u98ce\u683c, \u548c\u8be6\u5c3d\u7684\u63d0\u4ea4\u65e5\u5fd7.                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195211\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : stupil <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u6709\u4eba\u641e\u4e00\u4e2a\u4ee3\u7801\u6d41\u7a0b\u56fe\uff0c \u6a21\u5757\u600e\u4e48\u5212\u5206\uff0c\u63a5\u53e3\u600e\u4e48\u8c03\u7528\uff0c\u7c7b\u600e\u4e48\u5d4c\u5957\uff0c \u4e00\u76ee\u4e86\u7136\u3002<\/p>\n<p>\u5f53\u7136\u8fd9\u6837\u641e\u7684\u8bdd\uff0c\u60f3\u55b7\u7684\u4eba\u66f4\u65b9\u4fbf\u4e86\u3002                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195212\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : Solace202 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u90a3\u4e48\u6709\u6ca1\u6709\u54ea\u4f4d\u6709\u6bd4\u8f83\u597d\u7684 code review \u7684\u6700\u4f73\u5b9e\u8df5\u5462\uff1f\u8bdd\u8bf4\u6211\u56db\u5e74\u591a\u4e86\u4e24\u5bb6\u516c\u53f8\u90fd\u6ca1 review \u8fc7\u3002\u3002\u3002\u4e8c\u7ebf\u57ce\u5e02                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195213\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : stoneabc <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8bed\u6cd5\u9519\u8bef\u548c\u4ee3\u7801\u89c4\u8303\u4e0d\u5e94\u8be5\u5728\u95e8\u7981\u5c31\u5361\u56de\u53bb\u4e48\u3002\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195214\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : fewok <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u6211\u89c9\u5f97\u5427\uff0c\u4ee3\u7801\u8d28\u91cf\u4e0d\u662f\u9760 code review \u6765\u63a7\u5236\u7684\uff0c\u5e94\u8be5\u5148\u641e\u65b9\u6848\u8bbe\u8ba1\uff0c\u628a\u5173\u8c03\u7528\u6d41\u7a0b\uff08\u6b63\u5411\u3001\u9006\u5411\uff09\uff0c\u5c24\u5176\u662f\u5f02\u5e38\u5904\u7406\uff0c\u6709\u6ca1\u6709\u628a\u573a\u666f\u8003\u8651\u5168\uff0c\u67d0\u4e9b\u573a\u666f\u8be5\u4e0d\u8be5\u5b9e\u73b0\uff0c\u80fd\u4e0d\u80fd\u6362\u4e2a\u65b9\u5f0f\u5b9e\u73b0\uff0c\u6709\u6ca1\u6709\u9519\u8bef\u5904\u7406\u7b49\u7b49\uff0c\u5c24\u5176\u662f\u72b6\u6001\u673a\u7684\u6d41\u8f6c\u548c\u63a7\u5236\uff0c\u8c01\u9a71\u52a8\u8c01\uff0c\u80fd\u4e0d\u80fd\u9a71\u52a8\u7b49\u3002<br \/>\u81f3\u4e8e\u8bed\u6cd5\u9519\u8bef\u8fd9\u4e9b\uff0c\u4f60\u53f8\u641e\u4e2a\u5355\u5143\u6d4b\u8bd5\uff0c\u9759\u6001\u4ee3\u7801\u626b\u63cf\u4e0d\u5c31\u53ef\u4ee5\u4e86\uff08\u8bed\u6cd5\u90fd\u9519\u8bef\u4e86\uff0c\u8bf4\u660e\u8fd9\u90fd\u6ca1\u8fd0\u884c\u8d77\u6765\u8fc7\uff09                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195215\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : southwolf <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8bed\u6cd5\u9519\u8bef? \u8fd9\u4e9b\u4e0d\u5e94\u8be5\u76f4\u63a5\u88ab lint\/CI \u5361\u6389\u4e48?                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195216\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : lxk11153 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             233 \u5c31\u662f\u8fd9\u6837\u7684 [doge]                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195217\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : Cbdy <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u5c42\u7ea7\u592a\u591a\u4e86\u5427\uff0ccode review \u53ef\u80fd\u66f4\u9002\u5408\u7ba1\u7406\u6241\u5e73\u7684\u5de5\u4f5c\u7ec4\u7ec7\u5427\uff0c\u6bd4\u5982\u5fd7\u613f\u8005\u5408\u4f5c\u5f00\u53d1\u5f00\u6e90\u8f6f\u4ef6                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195218\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : dustinth <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8bed\u6cd5\u9519\u8bef\u548c\u4ee3\u7801\u89c4\u8303\u90fd\u53ef\u4ee5\u901a\u8fc7\u5de5\u5177\u81ea\u52a8\u62e6\u4f4f; \u8fd8\u6709\u66f4\u591a\u7684\u6bd4\u5982\u662f\u5426\u65b9\u6cd5\u8fc7\u957f, \u5d4c\u5957\u5c42\u6b21\u592a\u591a\u8fd9\u6837 common pratice \u4e5f\u53ef\u4ee5\u901a\u8fc7\u5de5\u5177\u5b9e\u73b0; \u771f\u6b63\u5230\u4e86\u4e1a\u52a1\u903b\u8f91\u5c42\u9762, \u5982\u679c\u4e0d\u662f\u6709\u914d\u5957\u7684 pair programming \u5b9e\u8df5, code review \u4e00\u4e2a\u6807\u51c6\u57fa\u672c\u4e0d\u592a\u73b0\u5b9e;<\/p>\n<p>\u662f\u5426\u4f7f\u7528\u4e25\u683c\u7684 code review, \u53d6\u51b3\u4e8e\u4ee3\u7801\u6539\u52a8\u7684\u9891\u7387\u548c\u6b21\u6570: \u4e1a\u52a1\u53d8\u5316\u5feb\u901f\u9891\u7e41\u7684\u9886\u57df\u6216\u8005\u4ee3\u7801\u4ef7\u503c\u4e0d\u592a\u9ad8\u4f46\u662f\u53c8\u4e0d\u53ef\u6216\u7f3a\u7684\u9886\u57df, code review \u53ef\u4ee5\u9002\u5f53\u653e\u677e\u9650\u5236; \u5bf9\u4e8e\u6210\u719f\u7684\u4e1a\u52a1\u4ef7\u503c\u5927\u7684\u9886\u57df, \u53ef\u4ee5\u91c7\u7528\u4e25\u683c\u7684 code review, \u8fd9\u6837\u624d\u80fd\u5e26\u6765\u66f4\u9ad8\u7684\u6536\u76ca.                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195219\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : huifer <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             sonarqube \u5404\u4e2a\u8bed\u8a00\u5236\u5b9a\u89c4\u5219, \u6216\u8005\u6bcf\u4e2a\u9879\u76ee\u6307\u5b9a\u4e00\u4e2a\u89c4\u5219. \u5148\u8fc7\u9759\u6001\u68c0\u67e5\u5427. \u5bf9\u4e8e\u6ce8\u91ca, \u65b9\u6cd5\u540d\u79f0\u8fd9\u4e9b\u5f97\u9700\u8981\u4eba\u529b\u6210\u672c, \u6216\u8005\u516c\u53f8\u5185\u90e8\u5b58\u5728\u4e00\u4e2a\u8bcd\u6c47\u8868. \u9488\u5bf9\u4e00\u4e2a\u9879\u76ee\u6216\u8005\u6574\u4e2a\u4f01\u4e1a\u5185\u7684, \u901a\u8fc7\u7edf\u4e00\u8bcd\u6c47\u8868\u6765\u63d0\u9ad8\u53ef\u8bfb\u6027. \u5bf9\u4e8e\u65b9\u6cd5\u540d\u79f0, \u7c7b\u540d\u79f0 \u540c\u6837\u9002\u7528.                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195220\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : VDimos <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8bed\u6cd5\u9519\u8bef\u600e\u4e48\u8fd8\u80fd\u5230 CR \u8fd9\u4e00\u6b65                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195221\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : impl <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div> <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195222\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : cdlixucd <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @impl lol                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195223\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : laminux29 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             Code Review \u7684\u672c\u8d28\u662f\u8ba9\u5927\u4f6c\u5e26\u840c\u65b0\u3002\u8fd9\u91cc\u6709\u51e0\u4e2a\u95ee\u9898\uff1a<\/p>\n<p>1.\u8ba9\u5927\u4f6c\u5e26\u840c\u65b0\uff0c\u6d6a\u8d39\u4e86\u5927\u4f6c\u7684\u65f6\u95f4\uff0c\u5bf9\u6574\u4e2a\u516c\u53f8\u4ef7\u503c\u6765\u8bf4\uff0c\u503c\u5f97\u5417\uff1f\u800c\u4e14\u5927\u4f6c\u4f1a\u8ba4\u4e3a\u81ea\u5df1\u88ab\u62db\u6765\u5e72\u8fd9\u4e8b\uff0c\u5c5e\u4e8e\u81ea\u5df1\u7684\u5c97\u4f4d\u804c\u8d23\u5417\uff1f<\/p>\n<p>2.\u840c\u65b0\u7f3a\u4e4f\u57fa\u7840\u7406\u8bba\uff0c\u5927\u4f6c\u6559\u840c\u65b0\uff0c\u840c\u65b0\u80fd\u542c\u61c2\u5417\uff1f\u6559\u5b66\u6548\u7387\u9ad8\u5417\uff1f<\/p>\n<p>3.\u8981\u63d0\u9ad8\u840c\u65b0\u6c34\u5e73\uff0c\u8fd8\u4e0d\u5982\u8ba9\u840c\u65b0\u5e26\u85aa\u8131\u4ea7\u53bb\u5b66\u6821\u5b66\u4e60\u3002                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195224\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : illusionist <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u4ee3\u7801 review \u6b63\u5e38\u6765\u8bb2\uff0c\u5bf9\u591a\u4eba\u5408\u4f5c\u5f00\u53d1\u9879\u76ee\u662f\u5fc5\u8981\u7684\uff0c\u4f46\u662f\u5f88\u591a\u65f6\u5019\u4f1a\u51fa\u73b0\u6709+2 \u6743\u9650\u7684\u4eba\u770b\u4e0d\u61c2\u4ee3\u7801\uff0c\u770b\u61c2\u4ee3\u7801\u7684\u4eba\u6ca1\u6709\u52a0+2 \u6743\u9650\u3002\u800c\u4e14\u5f88\u5bb9\u6613\u8d70\u5165\u4e3a\u4e86 review \u800c review, \u6d41\u4e8e\u5f62\u5f0f\u3002\u6ca1\u5565\u597d\u7684\u89e3\u51b3\u65b9\u6848\uff0c\u53ea\u80fd\u8bf4\u7ba1\u7406\u51b3\u7b56\u5c42\u8981\u5934\u8111\u6e05\u9192\uff0c\u77e5\u9053\u4e3a\u5565 review\uff0c\u5f88\u591a\u65f6\u5019 review \u662f\u4e00\u79cd\u6743\u5229\u8fd0\u52a8\uff0c\u800c\u4e0d\u662f\u771f\u6b63\u4e3a\u4e86\u63d0\u9ad8\u8f6f\u4ef6\u8d28\u91cf\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195225\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : GreyYang <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             LGTM!                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195226\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : DoctorCat <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u5e38\u6001\u3002\u793e\u4f1a\u5026\u6020\u95ee\u9898\u89e3\u51b3\u4e0d\u6765\u3002\u6211\u89c9\u5f97 CodeReview \u7684\u6d4b\u91cd\u70b9\u5e94\u8be5\u653e\u5728\u5e08\u5f92\u4f20\u6388\u9636\u6bb5\u548c\u522b\u4eba\u4fb5\u72af\uff08\u52a8\uff09\u76f8\u5173\u8d23\u4efb\u4eba\u7684\u4ee3\u7801\u65f6\uff0c\u5f7c\u6b64\u6709\u4e2a\u6280\u672f\u4ea4\u6d41\uff0c\u5c3d\u529b\u964d\u4f4e\u6280\u672f\u98ce\u9669\u548c\u503a\u52a1\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195227\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : zion03 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             https:\/\/google.github.io\/eng-practices\/review\/                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195228\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : lecher <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u5e26\u7740\u4e1a\u52a1\u7406\u89e3 review\uff0c\u6211\u53f8\u8981\u6c42 code review \u7684\u76ee\u7684\u662f\u4e3a\u4e86\u907f\u514d\u5355\u70b9\uff0c\u5728\u4ee3\u7801\u5b9e\u73b0\u7684\u601d\u8def\u4e0a\u8ba9\u56e2\u961f\u8fbe\u6210\u4e00\u81f4\uff0c\u81f3\u5c11\u76f8\u5173\u4e1a\u52a1\u5f00\u53d1\u4eba\u5458\u8fbe\u6210\u4e00\u81f4\uff0creview \u4e4b\u540e\u610f\u5473\u7740\u8fd9\u4e2a\u6a21\u5757\u51fa\u4e86\u95ee\u9898\uff0c\u6211\u4e5f\u53ef\u4ee5\u8d1f\u8d23\u4fee\u3002\u8fd9\u6837\u51fa\u4e86\u4efb\u4f55\u95ee\u9898\uff0creview \u8fc7\u7684\u4eba\u90fd\u53ef\u4ee5\u76f4\u63a5\u4fee\u6539\u4ee3\u7801\uff0c\u800c\u4e0d\u662f\u63a8\u8131\u8fd9\u4e2a\u6a21\u5757\u4e0d\u662f\u6211\u5199\u7684\uff0c\u6211\u4e0d\u9700\u8981\u4e3a\u6b64\u8d1f\u8d23\u3002<br \/>\u6240\u4ee5\u5728\u6211\u53f8\uff0ccode review \u6ca1\u6709\u628a\u91cd\u70b9\u653e\u5728\u8bed\u6cd5\u4e4b\u7c7b\u7684\u5b9e\u73b0\u7ec6\u8282\u4e0a\uff0c\u91cd\u70b9\u662f\u5728\u4ee3\u7801\u6709\u6ca1\u6709\u4f53\u73b0\u5408\u7406\u7684\u8bbe\u8ba1\u601d\u8def\uff0c\u4ee3\u7801\u53ef\u8bfb\u6027\u4e0a\u3002<br \/>\u800c\u4e14 code review \u5df2\u7ecf\u662f\u6700\u540e\u4e00\u6b65\u4e86\uff0c\u529f\u592b\u5e94\u8be5\u5728 code review \u4e4b\u5916\u3002<\/p>\n<p>\u6982\u8981\u8bbe\u8ba1\u6587\u6863\uff0c\u7ec6\u8282\u8bbe\u8ba1\u6587\u6863\uff0c\u90fd\u9700\u8981\u6709\u914d\u5957\u7684\u8bc4\u5ba1\uff0c\u5c06\u6280\u672f\u503a\u52a1\u5c3d\u53ef\u80fd\u51cf\u5c11\u3002\u8bbe\u8ba1\u6587\u6863\u7684\u8bc4\u5ba1\u901a\u5e38\u662f\u8001\u5e26\u65b0\uff0c\u8001\u624b\u5199\u6982\u8981\u8bbe\u8ba1\u6846\u4f4f\u4e1a\u52a1\u8fb9\u754c\uff0c\u65b0\u624b\u5199\u8be6\u7ec6\u8bbe\u8ba1\u660e\u786e\u5b9e\u73b0\u7ec6\u8282\uff0c\u7531\u8bc4\u5ba1\u6765\u4fdd\u8bc1\u76f8\u5173\u4eba\u5458\u5bf9\u4e1a\u52a1\u9700\u6c42\u7684\u62c6\u89e3\u7406\u89e3\u8fbe\u5230\u4e00\u81f4\u3002<\/p>\n<p>\u6b64\u5916\u5c31\u662f\u914d\u5957\u7684 CI lint test \u7b49\u652f\u6301\uff0c\u89c4\u8303\u4ee3\u7801\u98ce\u683c\uff0c\u4fdd\u8bc1\u529f\u80fd\u6ee1\u8db3\u9700\u6c42\u3002<\/p>\n<p>\u5b9e\u9645 review \u7684\u65f6\u5019\uff0c\u4f1a\u5feb\u901f\u770b\u770b test \u7684\u8986\u76d6\u7387\u5feb\u901f\u786e\u8ba4\u5bf9\u5916\u66b4\u9732\u7684\u8c03\u7528\u7ea6\u5b9a\u662f\u5426\u5408\u7406\uff0c\u8fb9\u754c\u6837\u4f8b\u662f\u5426\u8986\u76d6\u4e1a\u52a1\u9700\u6c42\u7684\u8981\u6c42\u3002\u5176\u6b21\u662f\u770b\u4ee3\u7801\u7ed3\u6784\uff0c\u53ef\u8bfb\u6027\u5982\u4f55\uff0c\u518d\u6b21\u624d\u662f\u8bed\u6cd5\u7ec6\u8282\uff0c\u6709\u6ca1\u6709\u4e0d\u5408\u7406\u7684\u5199\u6cd5\uff0c\u5bf9\u4e8e\u4e0d\u5408\u7406\u7684\u5199\u6cd5\u901a\u5e38\u53ea\u6709\u65b0\u4eba\u7684\u524d\u51e0\u4e2a PR \u9ad8\u6807\u51c6\u8bc4\u5ba1\uff0c\u751a\u81f3\u53ef\u80fd\u6709\u4e00\u4e2a\u65b0\u4eba\u4e09\u56db\u4e2a\u8001\u4eba\u8bc4\u5ba1\u7684\u60c5\u51b5\u3002<\/p>\n<p>\u8fd9\u4e48\u5e26\u51e0\u8f6e\u4e0b\u6765\uff0c\u5927\u90e8\u5206\u56e2\u961f\u6210\u5458\u7684\u4ee3\u7801\u98ce\u683c\u548c\u4e1a\u52a1\u62c6\u89e3\u90fd\u63a5\u8fd1\u4e00\u81f4\uff0c\u57fa\u672c\u4e0a\u51fa\u95ee\u9898\u90fd\u53ef\u4ee5\u8ba9\u4e00\u4e2a\u4eba\u4e00\u6746\u5b50\u6345\u5230\u5e95\u5c42\u5b9e\u73b0\u53bb\u5b9a\u4f4d\u548c\u4fee\u590d\u3002\u5982\u679c\u53d1\u73b0\u662f\u4ee3\u7801\u5206\u5c42\u548c\u67b6\u6784\u6ee1\u8db3\u4e0d\u4e86\u65b0\u589e\u9700\u6c42\u53ef\u80fd\u8fd8\u6709\u91cd\u6784\u7684\u60c5\u51b5\u3002<\/p>\n<p>\u4eab\u53d7\u5230\u7684\u6536\u76ca\u662f\uff1a<br \/>test \u662f\u4ee5\u4e1a\u52a1\u8fb9\u754c\u4e3a\u8981\u6c42\u5199\u7684\uff0c\u4e0d\u7528\u62c5\u5fc3\u91cd\u6784\u7684\u5f71\u54cd\u8303\u56f4\u8fc7\u5927\u3002<br \/>\u5e95\u5c42\u6a21\u5757\u4e1a\u52a1\u7a33\u5b9a\uff0c\u5bb9\u6613\u4ea4\u63a5\uff0c\u65b0\u4eba\u4e0a\u624b\u9605\u8bfb\u4ee3\u7801\u548c\u6539\u4ee3\u7801\u7684\u6210\u672c\u4f4e\u3002<\/p>\n<p>\u8fd9\u4e48\u505a\u7684\u95ee\u9898\u662f\uff0c\u529f\u80fd\u8fed\u4ee3\u901f\u5ea6\u592a\u6162\uff0c\u4e00\u4e2a\u4e1a\u52a1\u9700\u6c42\u4ece\u8bbe\u8ba1\u5230\u5b9e\u73b0\u62d6\u7684\u5468\u671f\u4f1a\u5f88\u957f\u3002\u8fd8\u4f1a\u6709\u4e0d\u5c11\u65f6\u95f4\u653e\u5728\u91cd\u6784\u507f\u8fd8\u6280\u672f\u503a\u4e0a\u3002<br \/>\u8fd9\u79cd\u6a21\u5f0f\u53ea\u9002\u5408\u6838\u5fc3\u4e1a\u52a1\uff0c\u5bf9\u4e8e\u5f00\u53d1\u5468\u671f\u6025\u8feb\u7684\u4efb\u52a1\u5e76\u4e0d\u592a\u9002\u5408\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195229\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : feather12315 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @lecher #20 \u65b0\u624b\u4ee3\u7801\u90fd\u641e\u4e0d\u61c2\uff0c\u548b\u5199\u5b9e\u73b0\u7ec6\u8282\uff1f                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195230\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : wobuhuicode <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u65b0\u5165\u804c\u7684\u516c\u53f8\u4e5f\u662f\u8fd9\u6837\u2026\u2026 review \u548c \u6ca1 review \u4e00\u4e2a\u6837\uff0c\u90fd\u662f\u4e3a\u4e86\u73b0\u5b9e\u4e0a\u7ea7\u6709\u5e72\u6d3b\u7684\u53d1\u4e2a \u5df2\u9605\u3002<br \/>\u4e0a\u7ebf\u4e4b\u540e\u4f9d\u65e7\u4e00\u5806 bug                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195231\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : zpxshl <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8bf4\u8bf4\u6211\u53f8\u60c5\u51b5<br \/>1 cr \u4e0d\u9700\u8981\u9886\u5bfc\uff0b 2\uff0c\u4e8b\u5b9e\u4e0a\u4e00\u822c\u4e5f\u4e0d\u4f1a\u627e\u9886\u5bfc cr<br \/>2 \u6539\u52a8\u5230\u7279\u5b9a\u6a21\u5757\u5f3a\u5236\u9700\u8981\u6a21\u5757\u8d1f\u8d23\u4eba cr<br \/>3 \u4ee3\u7801\u4e0d\u7b26\u5408\u8bed\u6cd5\u89c4\u8303\u7684\u4ee3\u7801\u63d0\u4ea4\u65f6\u81ea\u52a8\u62e6\u4f4f<\/p>\n<p>\u603b\u4f53\u8fd8\u884c\uff0c\u6bd5\u7adf\u6a21\u5757\u8d1f\u8d23\u4eba\u4e5f\u4e0d\u5e0c\u671b\u4f60\u7684\u6539\u52a8\u5f71\u54cd\u5230\u4ed6\u7684\u6a21\u5757\u3002\u81f3\u4e8e\u81ea\u5df1\u6a21\u5757\u5199\u7684\u4ee3\u7801\u627e\u4eba cr\uff0c\u4e3b\u8981\u770b\u5bf9\u65b9\u662f\u5426\u8d1f\u8d23\uff0c\u8d1f\u8d23\u7684\u4f1a\u6162\u6162\u770b\u4ed4\u7ec6\u770b\uff0c\u540c\u65f6\u6211\u4e5f\u4f1a\u5728\u65c1\u8fb9\u8bb2\u89e3\u3002 \u4e0d\u8d1f\u8d23\u7684\u76f4\u63a5\u5c31\uff0b 2 \u4e86\u3002 \u5446\u51e0\u4e2a\u6708\u4f60\u4e5f\u77e5\u9053\u8c01\u9760\u8c31\u8c01\u4e0d\u9760\u8c31\u4e86\u3002<\/p>\n<p>\u5f53\u7136\u4e86\uff0ccr \u8005\u5176\u5b9e\u662f\u65e0\u507f\u5e2e\u4f60\uff0c\u7528\u7231\u53d1\u7535\u4e86\uff0c\u6240\u4ee5\u627e\u4eba cr \u65f6\u6001\u5ea6\u597d\u70b9\uff0c\u5927\u7684\u6539\u52a8\u5c3d\u91cf\u5206\u6279 cr\uff0c\u591a\u70b9\u6ce8\u91ca\uff0c\u753b\u4e2a\u6d41\u7a0b\u56fe\u5c31\u66f4\u597d\u4e86\u3002\u4e00\u6765\u5c31\u4e0a\u5343\u884c\u4e1a\u52a1\u4ee3\u7801\u7684\u63d0\u4ea4\uff0c\u53cd\u6b63\u6211\u662f\u6ca1\u5fc3\u60c5\u770b\u5f97\u3002\u3002\u3002                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195232\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : 0Shaka <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u6bd4\u8f83\u597d\u7684\u5b9e\u8df5\u5c31\u662f\u5e73\u65f6\u53ea\u627e\u7ec4\u91cc\u4e00\u4e24\u4e2a\u4eba review, \u9ad8\u6548\u5feb\u901f\u8fd8\u6ca1\u6709\u90a3\u4e48\u591a b \u4e8b.                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195233\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : ga6840 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u4e2a\u4eba\u8ba4\u4e3a\u516c\u53f8\u91cc\u7684\u4ee3\u7801\u4e5f\u5e94\u8be5\u6709 maintainer\uff0c\u800c\u4e14\u4ed6\u548c\u4ed6\u7ef4\u62a4\u7684\u4ee3\u7801\u80af\u5b9a\u662f\u5229\u76ca\u76f8\u5173\u3002\u4f60\u7684 PR \u80af\u5b9a\u5e94\u8be5\u662f\u8fd9\u4e9b\u4eba review \u3002\u65e0\u5173\u7684\u6216\u8005\u5e73\u7ea7\u7684\u518d\u6216\u8005\u9ad8\u51fa\u51e0\u5c42\u7684\u9886\u5bfc\u6765\u770b\u7684\u8bdd\u6700\u591a\u597d\u5fc3\u770b\u770b\u3002\u5982\u679c\u7ec4\u7ec7\u6ca1\u6709\u4ec0\u4e48\u5956\u52b1\u70ed\u5fc3\u7684\u5236\u5ea6\uff0c\u90a3\u4e48\u516b\u6210\u4e5f\u5c31\u6ca1\u6709\u4eba\u6709\u610f\u613f\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195234\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : linux40 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8058\u51e0\u4e2a\u641e\u7a0b\u5e8f\u5206\u6790\u3001\u8bed\u4e49\u5206\u6790\u7684\u5427\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195235\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : lecher <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @feather12315 \u7279\u522b\u65b0\u7684\u65b0\u624b\u6311\u51e0\u4e2a\u5199\u597d\u7ec6\u8282\u8bbe\u8ba1\u7684\u6587\u6863\u8ba9\u4ed6\u5199\u5b9e\u73b0\uff0c\u5c3d\u53ef\u80fd\u8c03\u65b0\u589e\u4e1a\u52a1\u6216\u8005\u7279\u522b\u7b80\u5355\u7684 bugfix                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195236\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : MrGba2z <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             CR \u8fd9\u4ef6\u4e8b\u60c5\u6211\u89c9\u5f97\u4ece\u6548\u7387\u63d0\u5347\u4e0a\u6765\u8bf4\u7684\u4f18\u5148\u7ea7\u662f:<br \/>1. \u597d\u7684 infrastructure: \u6211\u4eec\u516c\u53f8\u4ee3\u7801\u7684\u81ea\u52a8\u68c0\u67e5\u4e0d\u4ec5\u4ec5\u8d1f\u8d23\u8bed\u6cd5, \u89c4\u8303, \u8fd8\u4f1a\u81ea\u52a8\u4f18\u5316 \u6761\u4ef6\u8868\u8fbe\u5f0f (\u6bd4\u5982\u4e00\u5806 XOR \u5982\u679c\u53ef\u80fd\u7684\u8bdd\u4ed6\u4f1a\u6539\u6210\u66f4\u77ed\u66f4\u5408\u7406\u7684\u5f62\u5f0f) \u4ee5\u53ca best practice \u548c deprecated \u7684\u81ea\u52a8\u4fee\u590d. \u6700\u540e\u4f1a\u8dd1\u914d\u7f6e\u597d\u7684\u6d4b\u8bd5. \u8fd9\u4e9b\u5168\u901a\u8fc7\u4e4b\u540e\u624d\u4f1a\u53d1\u9001 CR \u8bf7\u6c42, reviewers \u90fd\u4f1a\u6536\u5230\u63a8\u9001\u63d0\u793a (macOS \u7684\u8bdd\u5c31\u662f\u7cfb\u7edf\u901a\u77e5, \u4e5f\u6709 Chrome \u63d2\u4ef6\u4f1a\u6301\u4e45\u76d1\u89c6)<br \/>2. Reviewer \u4eba\u5458\u7684\u9009\u62e9, \u6bcf\u4e2a\u4eba review \u4ee3\u7801\u7684\u4e25\u683c\u7a0b\u5ea6\u662f\u4e0d\u4e00\u6837, \u5982\u679c\u4f60\u60f3\u5feb\u4e00\u70b9, \u5c31\u9009\u5df2\u77e5\u7684\u6bd4\u8f83\u677e\u7684\u4eba, \u5982\u679c\u60f3\u63d0\u9ad8\u6216\u8005\u5bf9\u81ea\u5df1\u5199\u7684\u4ee3\u7801\u6ca1\u4fe1\u5fc3, \u5c31\u627e review \u4e25\u683c\u7684\u4eba.<br \/>3. \u7ec4\u91cc review \u4ee3\u7801\u7684\u98ce\u683c. \u6211\u4eec TL \u4e4b\u524d\u4ecb\u7ecd\u8fc7\u4ed6\u7684\u98ce\u683c, \u4e0d\u4e00\u5b9a\u9002\u5408\u6240\u6709\u4eba\u4f46\u662f\u6211\u89c9\u5f97\u633a\u4e0d\u9519, \u4e3b\u8981\u7684\u51e0\u70b9\u662f: 1. \u5982\u679c\u4ee3\u7801\u8fbe\u5230\u4e86\u76ee\u7684, LGTM. 2. \u53ef\u8bfb\u6027 &gt; \u8bed\u8a00\u89c4\u8303 3. \u5c0f CR &gt; \u5927 CR \u4f46\u662f\u5982\u679c \u5927 CR \u5df2\u7ecf\u5199\u597d\u4e86, \u9664\u975e\u81ea\u5df1\u770b\u4e0d\u61c2, \u5426\u5219\u5c3d\u91cf\u4e0d\u8981\u8981\u6c42\u5bf9\u65b9\u62c6\u5206. \u9664\u4e86\u8fd9\u4e2a\u4e4b\u5916, \u6211\u4eec\u7ec4\u57fa\u672c\u4e0a\u6536\u5230 CR \u4f1a\u5728 5~30 \u5206\u949f\u5185\u5b8c\u6210 Review. (\u6ca1\u4eba\u8981\u6c42\u8fd9\u4e48\u505a, \u53ea\u662f\u5927\u5bb6\u90fd\u8fd9\u6837, unblock \u522b\u4eba\u4e5f\u662f\u95f4\u63a5\u63d0\u5347\u6574\u4e2a\u7ec4\u7684\u6548\u7387)<br \/>4. \u5c31\u662f\u81ea\u5df1\u5199\u7684\u4ee3\u7801\u4e86, CR \u63cf\u8ff0\u662f\u5426\u80fd\u63d0\u4f9b\u8db3\u591f\u7684\u80cc\u666f? CR \u662f\u5426\u592a\u5927\u800c\u4e0d\u65b9\u4fbf Review? (\u6bd4\u5982\u6211\u770b\u5230\u4e2a 2000 \u884c\u7684 CR, \u672c\u80fd\u7684\u4f1a\u60f3 \u8fd9 TM \u5f97\u627e\u4e2a 30 \u5206\u949f\u7684\u65f6\u95f4\u6bb5\u6162\u6162\u770b, \u73b0\u5728\u6ca1\u7a7a \u9a6c\u4e0a\u8981\u5403\u996d\u4e86, \u7136\u540e\u5c31&#8230;..) <\/p>\n<p>\u5f53\u7136\u8bdd\u8bf4\u56de\u6765,\u5f88\u591a\u4e8b\u60c5\u4e0d\u662f\u81ea\u5df1\u80fd\u6539\u53d8\u7684. \u5982\u679c\u4f60\u662f TL \u5012\u662f\u53ef\u4ee5\u5c1d\u8bd5\u6539\u53d8. \u5982\u679c\u6709\u673a\u4f1a\u7684\u8bdd, \u53bb\u5927\u516c\u53f8\u53bb\u54ea\u6015\u5b9e\u4e60\u4e00\u4e2a\u6708, \u4e5f\u80fd\u770b\u5230\u522b\u4eba\u7684\u8fd9\u5957\u6d41\u7a0b\u6709\u591a\u597d.                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195237\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : DamienS <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8fd9\u4e9b\u95ee\u9898\u4e0d\u96be\u89e3\u51b3\u5427<br \/>\u79ef\u6781\u6027:<br \/>1. \u4ee3\u7801 review \u52a0\u5165\u5230\u5b63\u5ea6\u5458\u5de5 kpi<br \/>2. review kpi \u5305\u542b\u8d28\u91cf\u8003\u6838<\/p>\n<p>\u89c4\u8303\uff1a<br \/>1. \u89c4\u8303 UI PR \u8981\u622a\u56fe\uff0c\u89c4\u8303\u65e5\u5fd7\u8981\u5199\u5565<br \/>2. \u89c4\u5b9a\u4e00\u4e9b\u8c01\u53ef\u4ee5\u63a5\uff0c\u51e0\u4e2a\u4eba\u63a5\u4e86\u5c31\u53ef\u4ee5\uff08\u4e00\u822c\u4e00\u4e2a\u4eba\u63a5\u4e86\u5c31\u884c\u5427\uff0c\u4e0d\u7136\u5f71\u54cd\u6548\u7387\uff0c\u800c\u4e14\u5199 c++\u7684 review java \u4ee3\u7801\u8fd9\u79cd\u5c31\u5f88\u626f\u6de1\uff0c\u7ecf\u7406\u63a5\u4e5f\u5f88\u626f\u6de1\uff09<br \/>3. \u89c4\u5b9a\u6240\u6709\u7684\u8bc4\u8bba\u4e0b\u4e2a\u7248\u672c\u5c31\u8981\u5168\u6539\u5b8c\uff0c\u4e0d\u80fd\u6539\u4e00\u534a\uff0c\u522b\u4eba\u53c8\u6307\u51fa\u6709\u7684\u8fd8\u6ca1\u6539<\/p>\n<p>\u7b80\u5316\u5de5\u7a0b\u5e08\u8d1f\u62c5\uff1a<br \/>1. \u6bcf\u4e2a PR \u81ea\u52a8\u5316\u76d1\u6d4b\u4e00\u4e9b\u57fa\u672c\u9519\u8bef\uff0c\u6bd4\u5982\u7528 linter\uff0c\u6216\u8005\u7aef\u5bf9\u7aef\u6d4b\u8bd5\uff0c\u6ca1\u8fc7\u7684\u56de\u53bb\u6539\uff0c\u76f4\u63a5\u4e0d\u5230 review \u90a3\u4e00\u6b65<br \/>2. \u5efa\u7acb\u4e00\u4e9b review group \u3002\u4e0d\u7528\u4e00\u4e2a\u4e00\u4e2a\u52a0\u6216\u8005\u51cf                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195238\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : laike9m <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8fd9 review \u5c31\u79bb\u8c31\u3002\u8fd8\u662f\u591a\u5b66\u5b66\u6b63\u89c4\u7684\u5927\u516c\u53f8\u662f\u600e\u4e48\u505a review \u7684\u5427                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195239\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : DamienS <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @MrGba2z CR..\u662f\u4e9a\u9ebb\u8001\u54e5\u4e48                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195240\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : meetubyaccident <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u4ee3\u7801\u6ca1\u6ce8\u91ca\uff0c\u968f\u4fbf\u770b\u770b\u5c31\u884c\uff0cTM \u8d39\u65f6\u95f4\u8fd8\u8981\u731c\u4f60\u4e3a\u5565\u4f60\u8fd9\u4e48\u5199\uff0c\u5f88\u7d2f\u4eba\u7684\uff0c\u81ea\u5df1\u7684\u6d3b\u4e0d\u7528\u5e72\u5417\uff1f\u51fa\u9519\u81ea\u5df1\u6539\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195241\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : yaaaaaak <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8bed\u6cd5\u95ee\u9898 ci \u90fd\u8fc7\u4e0d\u53bb\u5427                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195242\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : leavic <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u90fd\u4e00\u6837\uff0c\u505a\u786c\u4ef6\u5341\u51e0\u5e74\uff0c\u65e9\u5c31\u77e5\u9053\u4e86\u4e00\u4e2a\u57fa\u672c\u5b9a\u5f8b\uff1a\u6240\u6709\u4eba\u90fd\u4e0d\u9760\u8c31\uff0c\u4e0d\u8981\u6307\u671b\u522b\u4eba\u5e2e\u4f60\u68c0\u67e5\u51fa\u4efb\u4f55\u9519\u8bef\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195243\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : hpeng <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u90a3\u4e0d\u662f\u3002\u90a3\u4e9b\u90fd\u53eb\u4ee3\u7801\u5408\u5e76\u5de5\u7a0b\u5e08\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195244\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : b00tyhunt3r <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @wuzhouhui <br \/>\u8001\u54e5 \u60f3\u95ee\u4e00\u4e0b\u8be6\u7ec6\u7684\u63d0\u4ea4\u65e5\u5fd7\u5177\u4f53\u662f\u6307\u5565\uff0cgit commit \u7ed1\u5b9a\u5230\u7f16\u8f91\u5668\u5417                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195245\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : hantsy <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @kpppp \u53ea\u80fd\u8bf4\u4f60\u516c\u53f8\u7684\u5b9e\u65bd\u95ee\u9898\u5427\u3002<\/p>\n<p>\u53ef\u4ee5\u770b\u770b\u8fd9\u4e24\u4f4d\u7684\u89c2\u70b9\u3002<br \/>@lecher https:\/\/www.v2ex.com\/t\/725245#r_9776838<br \/>@huifer https:\/\/www.v2ex.com\/t\/725245#r_9776373<\/p>\n<p>\u5728\u7f16\u7801\u524d\uff0c\u5bf9\u4e8e\u4e00\u8d77\u8ba8\u8bba\u9700\u6c42\u7684\u7406\u89e3\uff0c\u6700\u7ec8\u786e\u5b9a\u4ee3\u7801\u8981\u5b8c\u6210\u54ea\u4e9b\u529f\u80fd\uff0c\u5728\u5f00\u53d1\u4e4b\u524d\u4e5f\u8981\u5f52\u7eb3\u51fa\u6765\uff0c\u53ef\u4ee5\u5728 PR \u6216\u8005 Issue \u5f62\u6210\u4e00\u4e2a Checklist\uff0c\u4f8b\u5982\u4e00\u4e2a\u8868\u5355\u6ce8\u518c\u3002<br \/>1 \u3002 \u5982\u679c\u4fe1\u606f\u586b\u5199\u4e0d\u5b8c\u6574\uff0cValidation \u5931\u8d25\uff0c\u63d0\u793a\u9519\u8bef<br \/>2 \u3002 \u4fe1\u606f\u5b8c\u6574\uff0c\u63d0\u4ea4\u540e\uff0c\u68c0\u6d4b Email\uff0c\u624b\u673a\u53f7\u662f\u5426\u5b58\u5728\uff0c\u5982\u679c\u5b58\u5728\uff0c\u53cd\u9988\u9519\u8bef\u4fe1\u606f<br \/>3 \u3002 \u5982\u679c\u4e00\u5207\u68c0\u6d4b\u901a\u8fc7\uff0c\u4fdd\u5b58\u7528\u6237\u4fe1\u606f\uff0c\uff08\u53d1\u9001 Welcome \u90ae\u4ef6\uff0c\u521d\u59cb\u4e00\u4e9b\u7528\u6237\u914d\u7f6e\u7b49\uff09<\/p>\n<p>\u9996\u5148\uff0c\u5bf9\u4e8e\u73b0\u4ee3\u8f6f\u4ef6\u5f00\u53d1\uff0c\u7279\u522b\u4e91\u73af\u5883\uff0c\u4e00\u5f00\u59cb\u5c31\u9700\u8981\u4e00\u4e2a\u81ea\u52a8\u5316\u7684 Pipeline \u914d\u7f6e\uff0c\u68c0\u6d4b\u5206\u652f\u4ee3\u7801\u60c5\u51b5\uff0c\u4e00\u76ee\u4e86\u7136\u3002<br \/>https:\/\/github.com\/hantsy\/spring-reactive-jwt-sample\/pull\/59<br \/>1\uff0cCommentlint \u7b49\uff0c<br \/>2\uff0c\u68c0\u6d4b\u8bed\u6cd5\uff08\u6bd4\u5982\u8001\u5f0f\u7684\u9759\u6001\u68c0\u6d4b Checkstyle, findbugs \u7b49\u5982\u679c\u4f60\u559c\u6b22\u53ef\u4ee5\u914d\u7f6e\u4e0b\uff0c\u751a\u81f3\u53ef\u4ee5\u52a0\u5165\u5230 Git Hooks \u91cc\u9762\u3002\u4e0d\u8fc7\u57fa\u4e8e\u4e91\u7684 SonarCloud \u68c0\u6d4b\u66f4\u8be6\u7ec6\uff0c\u6211\u4e0d\u518d\u7528\u672c\u5730\u9759\u6001\u5206\u6790\u5de5\u5177\uff09<br \/>3\uff0c\u8fd0\u884c\u6d4b\u8bd5\uff08\u5355\u5143\u6d4b\u8bd5\uff0c\u529f\u80fd \/\u96c6\u6210\u6d4b\u8bd5\uff09&lt;&#8212;&#8212;&#8211;\u5fc5\u4e0d\u53ef\u5c11\uff0c \u5982\u679c\u4e0d\u5199\u6d4b\u8bd5\uff0cCR \u53ea\u80fd\u662f\u81ea\u6b3a\u6b3a\u4eba\u3002<br \/>4, \u751f\u6210\u6d4b\u8bd5\u62a5\u544a\uff0c\u5305\u542b\u4ee3\u7801\u8d28\u91cf\u62a5\u544a\uff08 SonarCloud \u7684\u89c4\u5219\u662f Social \u66f4\u65b0\u7684\uff0c\u53ef\u4ee5\u652f\u6301\u6700\u65b0\u7684\u8bed\u6cd5\uff0c\u89c4\u8303\u7b49\u3002\u516c\u53f8\u5982\u679c\u5927\u65b9\uff0c\u53ef\u4ee5\u8d2d\u4e70 Code Climate\uff0c \u501f\u52a9\u4eba\u5de5\u667a\u80fd\u5206\u6790\u4ee3\u7801\uff0c\u53ef\u4ee5\u4fee\u6b63\u5f31\u667a\u4ee3\u7801\uff09\uff0c\u53ef\u4ee5\u770b\u5230 Bugs\uff0cCode Smell\uff0c\u6280\u672f\u503a\u52a1\u7b49<\/p>\n<p>\u90a3 CR \u7684\u65f6\u5019\uff0c\u6211\u4eec\u8981\u6e05\u7406\u6240\u6709\u5728\u8fd9\u4e2a PR \u4ea7\u751f\u7684\u6280\u672f\u503a\u52a1\uff0c\u4fee\u6b63 Bad Code Smell \u7b49\uff0c\u800c\u4e14\u8981\u68c0\u6d4b\u529f\u80fd\u96c6\u6210\u6d4b\u8bd5\u662f\u5426\u5305\u542b\u4e86\u6240\u6709\u7684\u8def\u5f84\uff0c\u662f\u5426\u6709\u6f0f\u6389\u4ec0\u4e48\u6b65\u9aa4\u7b49\u3002<\/p>\n<p>CR \u51e0\u4e4e\u4e0d\u9700\u8981 [\u4eba\u5de5\u65b9\u5f0f] \u5173\u5fc3\u4e00\u4e9b\u679d\u8282\u7ec6\u8282\uff0c\u4ec0\u4e48\u8bed\u6cd5\uff0c\u547d\u540d\u7b49\uff0c\u8fd9\u4e9b\u5de5\u5177\u68c0\u6d4b\uff08\u5728 CI \uff09\u90fd\u53ef\u4ee5\u5b8c\u6210\u3002\u66f4\u591a\u7684\u76ee\u7684\u5927\u5bb6\u4e00\u4e9b\u8ba8\u8bba\u548c\u5206\u4eab\uff0c\u771f\u6b63\u505a\u5230 Pair programming \u548c Brainstorming \u3002                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195246\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : andj4cn <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u6211\u89c9\u5f97\u8ddf\u5f00\u53d1\u56e2\u961f\u5c42\u7ea7\u7ba1\u7406\u6709\u5173\u3002\u5f00\u53d1\u56e2\u961f\u5e94\u5c3d\u53ef\u80fd\u6241\u5e73\u5316\uff0c\u6211\u4e00\u822c\u4ee3\u7801\u53ea\u88ab 2 \u4e2a\u4eba review\uff0c\u6700\u591a\u4e86\u3002\u5176\u4ed6\u4eba\u60f3 review \u4e5f\u53ef\u4ee5                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195247\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : ClericPy <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u54c8\u54c8\u54c8 \u5176\u5b9e\u633a\u7fa1\u6155\u6709\u6b63\u7ecf code review \u7684\u516c\u53f8\u7684, \u73b0\u5728\u5929\u5929\u8d76\u8fdb\u5ea6\u6587\u6863\u90fd\u8ddf\u4e0d\u4e0a, \u522b\u8bf4 review \u4e86<\/p>\n<p>PS: \u542c\u8bf4\u6709\u4e00\u79cd\u8bbe\u8ba1\u5c31\u662f\u8ba9\u4e0d\u61c2\u8fd9\u95e8\u8bed\u8a00\u7684\u4eba\u4e5f\u80fd\u770b\u61c2\u4f60\u4ee3\u7801\u7684, \u7c7b\u4f3c\u4e8e\u6ce8\u91ca\u8a00\u7b80\u610f\u8d45\u5730\u505a\u5230\u7c7b\u4f3c\u5c0f\u9ec4\u9e2d\u7684\u7a0b\u5ea6                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195248\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : charlie21 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u54c8\u54c8 \u201c\u4e1a\u52a1 review\u201d \u6bd4 \u201ccode review\u201d \u66f4\u91cd\u8981\u3002\u5149 review code \u6ca1\u5565\u610f\u601d                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195249\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : MrGba2z <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @DamienS \u4e0d\u662f \u8fd9\u91cc\u53ea\u662f\u7528\u4f5c Code Review \u7684\u7b80\u5199.                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195250\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : loveyou1 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             review \u6bdb\u7ebf\uff0cprettier + \u5404\u79cd\u8bed\u8a00 lint + \u7f16\u8f91\u5668\u63d2\u4ef6\uff0c\u628a\u6700\u57fa\u672c\u7684\u4e1c\u897f\u5b9a\u6b7b\uff0c\u5c31\u662f\u6700\u597d\u7684\u529e\u6cd5\uff0c\u4e00\u5927\u5806\u4eba\u6ca1\u641e\u8fd9\u4e2a\u4e1a\u52a1\u7684\u6765 review \u8fd9\u4e2a\u4ee3\u7801\uff0c\u903b\u8f91\u4e0d\u90fd\u4e0d\u6e05\u695a\uff0c\u8fd8 review\uff0c\u5f00\u6e90\u7684\u9879\u76ee review \u4e5f\u591a\u6570\u662f\u901a\u8fc7\u8dd1\u81ea\u52a8\u5316\u811a\u672c review \u7684\uff0c\u4e00\u822c\u90fd\u4e0d\u4f1a\u53bb\u770b\u7684\uff0c\u6bcf\u4e2a\u4eba\u601d\u8def\u4e0d\u4e00\u6837\uff0c\u5904\u7406\u65b9\u5f0f\u4e5f\u4e0d\u4e00\u6837\uff0c\u5982\u679c\u6c34\u5e73\u5dee\u522b\u5927\u4e00\u4e9b\uff0c\u53ef\u4ee5\u6539\u4e00\u4e0b\u4e1c\u897f\uff0c\u5982\u679c\u6c34\u5e73\u5dee\u4e0d\u591a\uff0c\u4eba\u5de5 review \u6709\u6bdb\u7ebf\u7528\uff0c\u8fd8\u4e0d\u5982\u5206\u4eab\u5206\u4eab\u5b9e\u73b0\u601d\u8def\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195251\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : firefox12 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             cr \u7684\u5173\u952e\u662f\u4e1a\u52a1\u592a\u5feb\uff0c\u6ca1\u65f6\u95f4\u641e\uff0c\u4e00\u4e2a\u9700\u6c42\u53ef\u80fd\u6ca1\u4e0a\u7ebf\u5c31\u5df2\u7ecf\u6539\u4e86\u3002<\/p>\n<p>\u771f\u6b63\u597d\u7684\u662f java c++ \u7528\u4ec0\u4e48\u7248\u672c\u7684\u5de5\u5177\u5e2e\u4f60\u505a cr \u524d\u7684\u6821\u9a8c\uff0c \u90a3\u4e9b\u89c4\u5219\uff0c\u8fd9\u4e9b\u66f4\u6709\u843d\u5730\u7684\u5fc5\u8981\u3002                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195252\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : yuekcc <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @loveyou1 \u8d5e\u540c\u3002<\/p>\n<p>\u5206\u4eab\u601d\u8def\u6bd4\u8f83\u770b\u5177\u4f53\u4ee3\u7801\u5b9e\u7528\u5f97\u591a\u3002crud \u4e1a\u52a1\u573a\u666f\uff0c\u9010\u884c\u770b\u4ee3\u7801\u786e\u5b9e\u662f\u6d6a\u8d39\u6240\u6709\u4eba\u7684\u65f6\u95f4\uff0c\u800c\u4e14\u4ee3\u7801\u5199\u5f97\u4e0d\u597d\uff0c\u5f88\u5927\u53ef\u80fd\u5c31\u662f\u601d\u8def\u6ca1\u6709\u8f6c\u8fc7\u6765\u3002<\/p>\n<p>\u4e00\u822c\u4ee3\u7801\u95ee\u9898\uff0c\u5e94\u8be5\u901a\u8fc7\u5de5\u5177\u89e3\u51b3\u3002                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195253\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : crclz <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u8fd9\u91cc\u6709\u4e00\u7bc7\u6587\u7ae0\uff0c\u91cc\u9762\u5305\u542b\u4e86 Thoughtworks \u516c\u53f8\u505a code review \u7684\u90e8\u5206\u6d41\u7a0b\u3002<br \/>https:\/\/insights.thoughtworks.cn\/code-review\/<\/p>\n<p>\u6211\u6458\u6284\u4e00\u6bb5\uff1a<br \/>\u201c<br \/>\u4e3b\u6301\u4eba\u8bf4\uff1a\u201c\u6211\u4eec\u77e5\u9053\uff0c\u5982\u679c\u4ee3\u7801\u7f16\u5199\u5f97\u597d\uff0c\u90a3\u4e48\u4e0d\u662f\u4f5c\u8005\u7684\u4eba\u5c31\u80fd\u5728\u6ca1\u6709\u4f5c\u8005\u5e2e\u52a9\u7684\u60c5\u51b5\u4e0b\u8bfb\u61c2\u3002\u6211\u5e0c\u671b\u4e00\u4f4d\u4e0d\u662f\u8fd9\u6bb5\u4ee3\u7801\u4f5c\u8005\u7684\u5fd7\u613f\u8005\uff0c\u6765\u4e3a\u5927\u5bb6\u89e3\u91ca\u4e00\u4e0b\u8fd9\u6bb5\u4ee3\u7801\u662f\u505a\u4ec0\u4e48\u7684\u3002\u201d\u4e00\u4f4d\u975e\u4f5c\u8005\u7684\u5fd7\u613f\u8005\u4e0a\u6765\u9010\u884c\u89e3\u91ca\u4ee3\u7801\uff0c\u5e76\u56de\u7b54\u5927\u5bb6\u7684\u7591\u95ee\u3002<br \/>\u4e3b\u6301\u4eba\u7b49\u4ee3\u7801\u89e3\u91ca\u5b8c\u540e\uff0c\u95ee\u5927\u5bb6\uff1a\u201c\u8fd9\u6bb5\u4ee3\u7801\u5927\u5bb6\u8fd8\u6709\u770b\u4e0d\u61c2\u7684\u5730\u65b9\u5417\uff1f\u201d\u5982\u679c\u6709\u95ee\u9898\uff0c\u5305\u62ec\u4f5c\u8005\u5728\u5185\u7684\u53c2\u4f1a\u8005\u90fd\u53ef\u4ee5\u56de\u7b54\u95ee\u9898\uff0c\u4f46\u5927\u5bb6\u90fd\u4e0d\u63d0\u8c01\u662f\u4f5c\u8005\u3002<br \/>\u5927\u5bb6\u90fd\u770b\u61c2\u4ee3\u7801\u540e\uff0c\u4e3b\u6301\u4eba\u95ee\uff1a\u201c\u5927\u5bb6\u8bf4\u8bf4\u8fd9\u6bb5\u4ee3\u7801\u6709\u6ca1\u6709\u597d\u7684\u7f16\u5199\u6a21\u5f0f\u54b1\u4eec\u53ef\u4ee5\u7ee7\u7eed\u53d1\u626c\uff1f\u2026\u2026<br \/>\u201d<\/p>\n<p>\u4e3b\u8bf4\u770b\u4e0d\u61c2\u4ee3\u7801\uff0c\u90a3\u80af\u5b9a\u662f\u51fa\u4e86\u4e25\u91cd\u7684\u95ee\u9898\uff08\u5728\u7f16\u5199\u4ee3\u7801\u7684\u9636\u6bb5\uff09\u3002                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195254\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : asanelder <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @fewok #6 \u540c\u610f\uff0c\u4ffa\u8ba4\u4e3a review \u4e0d\u662f review \u8bbe\u8ba1\u76f8\u5173\uff0c\u8bbe\u8ba1\uff0c\u89c4\u8303\u4e4b\u7c7b\u7684\u5e94\u8be5\u4e4b\u524d\u5c31\u505a\u597d\u4e86\uff0creview \u53ea\u662f\u770b check in \u7684\u4ee3\u7801\u662f\u5426\u7b26\u5408\u7cfb\u7edf\u7684\u8bbe\u8ba1\u548c\u89c4\u8303\uff0c\u9632\u6b62\u574f\u5473\u9053\u7684\u4e1c\u897f\u8fdb\u6765\u3002<\/p>\n<p>\u5173\u952e\u95ee\u9898\u662f\uff0c\u5f88\u591a\u9879\u76ee\u8bbe\u8ba1\u548c\u89c4\u8303\u90fd\u6ca1\u6765\u7684\u53ca\u5236\u5b9a\uff0c\u5c31\u5f00\u59cb\u5199\u4ee3\u7801\u4e86\uff0c\u7136\u540e\u4e3a\u4e86 review \u800c review\uff0c\u7b80\u76f4\u662f\u81ea\u671f\u671f\u4eba\u7f62\u4e86\u3002<\/p>\n<p>\u8fd8\u6709\uff0c\u73b0\u5728\u7684\u4ea7\u54c1\u548c\u9879\u76ee\u7ecf\u7406\u8fd8\u662f\u8981\u61c2\u5f00\u53d1\u7684\u6d41\u7a0b\u7684\uff0c\u4e0d\u662f\u53ea\u5199\u4e1a\u52a1\u4ee3\u7801\u624d\u7b97\u662f\u6709\u4ea7\u51fa\uff0c\u90a3\u4e9b\u4e3a\u6b64\u652f\u6491\u7684\u4e1c\u897f\uff0c\u4e5f\u662f\u8981\u8ba4\u771f\u8003\u8651\uff0c\u52a0\u5165\u5230\u8fdb\u5ea6\u65f6\u95f4\u5f00\u53d1\u4e2d\u7684\u3002\u4f46\u4ffa\u89c2\u5bdf\u7684\u73b0\u8c61\u662f\uff0c\u4f3c\u4e4e\u4ed6\u4eec\u53ea\u8ba4\u4e3a\u53ea\u8981\u5f00\u53d1\u4e1a\u52a1\u4ee3\u7801\u5c31\u884c\u4e86\uff0c\u9884\u4f30\u65f6\u95f4\u4e2d\u6839\u672c\u6ca1\u6709\u4e3a\u5176\u5b83\u76f8\u5173\u7684\u7559\u51fa\u65f6\u95f4\u3002<\/p>\n<p>\u4f60\u8bf4\u4f60\u641e\u4e86 CI\/CD, \u641e\u4e86\u8bbe\u8ba1\uff0c\u81ea\u52a8\u5316\u4e86\u4e00\u4e9b\u6d41\u7a0b\uff0c\u5236\u5b9a\u5f00\u53d1\u5206\u652f\u89c4\u8303\u7b49\u7b49\u548c\u4e1a\u52a1\u4e0d\u76f4\u63a5\u76f8\u5173\u7684\u4e8b\uff0c\u5728\u4ed6\u4eec\u770b\u6765\u3002<\/p>\n<p>\u201c\u5c31\u662f\u6ca1\u4efb\u4f55\u4ea7\u51fa\u201d                                                            <\/p><\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195255\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : BIAOXYZ <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @impl #13 \u54c8\u54c8\uff0c\u8fc7\u4e8e\u771f\u5b9e\uff01\u5f15\u8d77\u8212\u9002\u3002\u3002\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195256\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : Zchary <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u6211\u6ca1\u731c\u9519\u7684\u8bdd\uff0c\u4f60\u8bf4\u7684\u5e94\u8be5\u662f\u83ca\u5382\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195257\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : lbyo <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @impl #13 LGTM <br \/>hhhhhh                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195258\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : ruzztok <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u6211\u516c\u53f8\u4ee3\u7801\u5f02\u5e38\u5904\u7406\u975e\u5e38\u5dee\u3002\u3002\u751f\u4ea7\u73af\u5883\u65e5\u5fd7\u6587\u4ef6\u6bcf\u79d2\u90fd\u5728\u6253\u5370\u9519\u8bef\u4fe1\u606f                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195259\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : qinyusen <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             https:\/\/www.v2ex.com\/t\/719560#reply24<\/p>\n<p>11 \uff0c\u548c 23 \uff0c \u867d\u7136\u9898\u76ee\u662f\u63d0\u9ad8\u81ea\u5df1\uff0c\u4f46\u662f\u5b9e\u9645\u4e0a\uff0ccode review \u7684\u65f6\u5019\uff0c\u9644\u4e0a\u8bbe\u8ba1\u6587\u6863\u7684\u5185\u90e8 wiki \u94fe\u63a5\u4e4b\u7c7b\u7684\u3002\u3002\u3002\u662f\u4e00\u4e2a\u5f88\u53cb\u597d\u7684\u505a\u6cd5\uff0c \u5f88\u5bb9\u6613\u8ba9\u5927\u4f6c\u6311\u523a\u3002<br \/>\u6700\u540e\u770b\u770b\u8bbe\u8ba1\u6587\u6863\u6ca1\u95ee\u9898\uff0c \u5b9e\u73b0\u5927\u7ea6 1\uff1a1 \u5c31\u53ef\u4ee5\u8fc7\u4e86\u3002                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195260\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : kuro1 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             LGTM                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195261\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : ytmsdy <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             @impl \u54c8\u54c8\u54c8\u54c8\uff0cfix_shit&#8230;                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li data-pid=\"4195262\" data-uid=\"2\">\n<div>\n<div>\n<div> <span>\u8cc7\u6df1\u5927\u4f6c : ksssdh123 <\/span>  <\/div>\n<div> <i title=\"\u5f15\u7528\"><\/i>  <span>          <\/span> <\/div>\n<\/p><\/div>\n<div>                                                             \u54e5\u4eec\uff0c\u4f60\u8bf4\u5f97\u592a\u68d2\u4e86~<br \/>\u4f46\u73b0\u5728\u5f80\u5f80\u6700\u53ef\u6015\u7684\u662f\u4ec0\u4e48\u4f60\u77e5\u9053\u5417?\u65b9\u6cd5\u8bba\u5927\u5bb6\u90fd\u77e5\u9053\uff0c\u4f46\u5f80\u5f80\u73b0\u5b9e\u5c31\u662f\u9677\u5165\u6b7b\u5faa\u73af\uff0c\u4e00\u8fb9\u558a\u7740\u8981\u5feb\u901f\u4e0a\u7ebf\uff0c\u4e00\u8fb9\u53c8\u8981\u6293\u8bc4\u5ba1\u3001\u6293\u6587\u6863\uff0c\u8001\u5de5\u7a0b\u5e08\u53c8\u4e0d\u613f\u5e26\u840c\u65b0\uff0c\u7136\u540e\u5f88\u591a\u6d41\u7a0b\u90fd\u662f\u6577\u884d\u4e86\u4e8b\uff0c\u9664\u4e86\u7f16\u7801\u5de5\u4f5c\uff0c\u4f46\u8fd9\u79cd\u73af\u5883\u4e0b\uff0c\u8fd9\u79cd\u4ea7\u7269\u4e5f\u5c31\u591f\u6f14\u793a&#8230;.<br \/>\u8ddf\u4e3b\u6240\u8bf4\u7684\u4e00\u6837\uff0c\u6d41\u7a0b\u53d8\u6210\u5f62\u5f0f\uff0c\u5f88\u591a\u90fd\u662f\u8d70\u9a6c\u89c2\u82b1                                                            <\/div>\n<\/p><\/div>\n<\/li>\n<li>\n","protected":false},"excerpt":{"rendered":"<p>\u6211\u53d1\u73b0 \u6211\u516c\u53f8\u7684 review c&hellip;<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":[],"categories":[],"tags":[],"_links":{"self":[{"href":"http:\/\/4563.org\/index.php?rest_route=\/wp\/v2\/posts\/201365"}],"collection":[{"href":"http:\/\/4563.org\/index.php?rest_route=\/wp\/v2\/posts"}],"about":[{"href":"http:\/\/4563.org\/index.php?rest_route=\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"http:\/\/4563.org\/index.php?rest_route=\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"http:\/\/4563.org\/index.php?rest_route=%2Fwp%2Fv2%2Fcomments&post=201365"}],"version-history":[{"count":0,"href":"http:\/\/4563.org\/index.php?rest_route=\/wp\/v2\/posts\/201365\/revisions"}],"wp:attachment":[{"href":"http:\/\/4563.org\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=201365"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"http:\/\/4563.org\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=201365"},{"taxonomy":"post_tag","embeddable":true,"href":"http:\/\/4563.org\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=201365"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}