search

it-kmitl-2018 / group3-backend

1 stars 0 forks source link

[Add] TradeDeliveryTerms, TradeTax, AssociatedDocumentLineDocument model & unit test #41

Closed Chiibi closed 6 years ago

Chiibi commented 6 years ago

Add model & unit test according to #33

champillon commented 6 years ago

คะแนนครับ 3.75 คะแนน x 2 Promotion => 7.5 คะแนน 5.1 การ design เป็น components เพื่อทำให้คนอื่นทำงานร่วมกันต่อได้ง่าย => 1 คะแนน 5.2 คุณภาพของการตั้งชื่อตัวแปร, ฟังก์ชั่น และความเรียบร้อยของ code => 0.5 คะแนน (ชื่อตัวแปรน่าเกลียด ถึงยังไม่สอนแต่ขอหักไว้ก่อน) 5.3 ความยากเชิงเทคนิคที่นำมาประยุกต์ใช้ เช่น แปลง XML Schema เป็น Java code => 1 คะแนน (ใช้ Lombok) 5.4 รายละเอียดการตั้งชื่อใน PR name และ commit log รวมทั้งรายละเอียดที่บรรยายใน issue ของ PR นั้นๆ => 1 คะแนน 5.5 ความสอดคล้องกันกับ PR อื่นๆของทีม ที่ทำงานร่วมกันได้ดี => 0.25 คะแนน (ทำให้ code สะอาด เพื่อนๆ maintain ง่าย)