១. សរសេរ Pull Request តូចៗជាងមុន
ជំនួសឱ្យដាក់ស្នើ PR ធំៗដែលមានបន្ទាត់កូដរាប់រយ ឬរាប់ពាន់ បង្ហាញការផ្លាស់ប្តូរតូចៗ ដែលផ្តោតលើអ្វីមួយច្បាស់លាស់។
ហេតុផល៖ PR តូចងាយឱ្យអ្នកពិនិត្យអាន យល់ និងសាកល្បង។ វាកាត់បន្ថយការចំរូងច្រឡំ និងជួយរកឃើញកំហុសមុននឹងចូលបញ្ចូល (merge)។
អនុសាសន៍ល្អ៖ ព្យាយាមរក្សា PR មិនឱ្យលើសប្រហែល 200 បន្ទាត់កូដ ហើយគួរតែទាក់ទងតែជាមួយមុខងារ ឬ bug fix តែមួយ។
២. អនុវត្ត PR Stacking
បំបែកការផ្លាស់ប្តូរធំៗជាបណ្ដុំ PR តូចៗ ដែលមួយៗស្ថាបនានៅលើមួយមុន។
ហេតុផល៖ អ្នកពិនិត្យអាចផ្តោតលើការផ្លាស់ប្តូរមានហេតុផលមួយម្តង ដោយមិនមានការច្របូកច្របល់។ វាក៏ងាយស្រួលក្នុងការត្រឡប់ក្រោយ ឬកែសម្រួលផ្នែកណាមួយ។
ឧទាហរណ៍៖ បើបន្ថែម Authentication Flow ថ្មី PR ដំបូងអាចបង្កើត Data Models បន្ទាប់មក PR បន្ទាប់ធ្វើ API Endpoints ហើយ PR ចុងក្រោយគឺភ្ជាប់ UI/Frontend។
៣. ផ្ដល់ព័ត៌មានបរិបទលម្អិត
សរសេរពន្យល់យ៉ាងច្បាស់នៅក្នុង PR Description។ ភ្ជាប់ Link ទៅ Ticket កូដ ឬឯកសាររចនា ហើយបន្ថែម Screenshot ឬ Diagram ប្រសិនបើមាន។
ហេតុផល៖ អ្នកពិនិត្យមិនចាំបាច់ទាយថាតើកូដមានគោលបំណងអ្វី។ ព័ត៌មានច្បាស់ជួយកាត់បន្ថយការសួរឆ្លើយថយក្រោយ និងបន្ថយពេលពិនិត្យ។
គន្លឹះ៖ ប្រើទម្រង់ What / Why / How
- What៖ តើអ្វីកំពុងតែផ្លាស់ប្តូរ?
- Why៖ ហេតុអ្វីបានត្រូវការផ្លាស់ប្តូរនេះ?
- How៖ តើអ្វីបានធ្វើឱ្យវាទទួលបាន?
៤. អនុវត្តសុវត្ថិភាពផ្នែកស្ថាបត្យកម្ម
រចនាមូលដ្ឋានកូដឱ្យមានភាពម៉ូឌុល និងការពារផ្នែកសំខាន់ៗ។ កំណត់អ្នកពិនិត្យដែលស្គាល់ល្អអំពីផ្នែកដែលផ្លាស់ប្តូរ។
ហេតុផល៖ ជៀសវាងការបំផ្លាញមុខងារសំខាន់ និងធានាបាននូវសមត្ថភាពស្មើគ្នានៅទូទាំង Codebase។
ឧទាហរណ៍៖ បើកូដប៉ះពាល់ប្រព័ន្ធ Authentication អ្នកពិនិត្យគួរមានចំណេះដឹងសុវត្ថិភាព។
៥. លើកទឹកចិត្តការសហការពេលវេលាពិត (Real-time Collaboration)
ប្រើការពិភាក្សាជាស្រេច Pair Programming ឬ Rapid Prototyping។
ហេតុផល៖ មតិយោបល់ភ្លាមៗកាត់បន្ថយចំនួន Iterations និងធ្វើឱ្យគុណភាពកូដប្រសើរឡើង។
ឧទាហរណ៍៖ អ្នកអភិវឌ្ឍ២នាក់ជួយគ្នាលើមុខងារលំបាកមួយ ជំនួសឱ្យបញ្ចូល PR ធំៗពេលក្រោយ។
៦. រួមបញ្ចូលជំនាញចម្រុះ
កំណត់អ្នកពិនិត្យទៅតាមជំនាញ Domain ផ្សេងៗ ដូចជា Front-end, Back-end, Database, Security។
ហេតុផល៖ ធានាថាការផ្លាស់ប្តូរត្រូវបានពិនិត្យដោយអ្នកដែលយល់ពីហានិភ័យ។
លទ្ធផល៖ កំហុសតិច និងចែករំលែកចំណេះដឹងជាក្រុមបានប្រសើរ។
៧. តុល្យភាពផ្នែកភារកិច្ចក្រុម
តាមដានចំនួន PR ដែលអ្នកនីមួយកំពុងពិនិត្យ ឬអភិវឌ្ឍ។ កុំឱ្យអ្នកណាម្នាក់ពិបាកពេក។
ហេតុផល៖ កាត់បន្ថយ Bottlenecks និងធ្វើឱ្យក្រុមមានផលិតភាពល្អ។ PR មិនស្នាក់ចោលយូរ។
គន្លឹះ៖ កំណត់ចំនួន PR បើកចំហមិនឱ្យលើសសម និងតាមដានពេលវេលាពិនិត្យឱ្យទាន់ហេតុការ។
៨. អនុវត្តស្វ័យប្រវត្តិកម្មទូទាំង Workflow
ប្រើឧបករណ៍ដូចជា GitHub Actions, Jenkins, ឬ CI/CD Pipelines ដើម្បីត្រួតពិនិត្យកូដដោយស្វ័យប្រវត្តិ រួមមាន Linting, Tests, Formatting, Security Checks។
ហេតុផល៖ ស្វ័យប្រវត្តិកម្មជួយរកកំហុសពីដំបូង បង្ខំស្ដង់ដារ និងកាត់បន្ថយការងារដោយដៃ។
ឧទាហរណ៍៖ PR នឹងបរាជ័យ Build ប្រសិនបើ Tests បរាជ័យ ដោយការពារកូដខូចមិនឱ្យ Merge ចូល។
ប្រសិនបើអ្នកចង់ ខ្ញុំអាចធ្វើជាផ្ទាំង cheat sheet ជាភាសាខ្មែរ ដើម្បីអ្នកនិងក្រុមប្រើក្នុង PR Workflow 🎯
អ្នកចង់ឱ្យខ្ញុំធ្វើវាដែរទេ SEANG?
Seng Seang Leng
Web Developer