जब आप किसी का React pull request review करते हैं तो क्या देखते हैं?

पहले data के बहाव का सहीपन: state कहां रहती है, effects जरूरी हैं या नहीं और उनकी dependencies ईमानदार हैं या नहीं, और keys स्थिर हैं या नहीं। फिर user को दिखने वाले किनारे, यानी loading, खाली और error की हालत, साथ ही keyboard और screen reader की पहुंच। फिर performance, सिर्फ वहां जहां वह नापी जा सके। शैली और नाम सबसे आखिर में, और जो कुछ कोई linter या formatter तय कर सकता है उस पर review में comment होना ही नहीं चाहिए।

मैं पहले ब्योरा और tests पढ़ता हूं, फिर diff, क्योंकि मैं यह जानना चाहता हूं कि बदलाव करना क्या चाहता है इससे पहले कि मैं आंकूं कि वह कैसे करता है। मेरा पहला दौर data के बहाव का है। यह state कहां रहती है, क्या वह नीचे रह सकती थी, वह effect सच में चाहिए या वह derived state है, dependencies ईमानदार हैं या किसी ने चुपचाप lint नियम दबा दिया है, और list की keys स्थिर हैं या नहीं। दूसरा दौर किनारों का है, और ज्यादातर bugs वहीं जाते हैं: loading के दौरान यह क्या दिखाता है, list खाली होने पर, और request फेल होने पर, और क्या मैं इसे keyboard से चला सकता हूं। तीसरा performance है, पर सिर्फ तब जब मैं किसी असली लागत पर उंगली रख सकूं, महज एहसास पर नहीं। मैं अपने comments को या तो रोकने वाला या बस एक ख्याल बताकर लिखता हूं, क्योंकि बिना label की मीनमेख pull requests को दिनों तक अटका देती है। और मैं formatting या imports के क्रम पर कभी comment नहीं करता; अगर वह मायने रखता है तो उसकी जगह मेरी राय नहीं, linter है।