AlfianHanifFY / mini-project-be-information

0 stars 0 forks source link

Review #1 #1

Open gibranfsh opened 3 months ago

gibranfsh commented 3 months ago

Overall udah bagus 😁👍👍, namun kalo bisa commit-nya dikit-dikit yaa supaya gampang di-track dan juga commit message-nya kalo bisa dibuat agar developer lain mengerti apa yang kamu commit.

Misalnya, commit message kamu kan ada yang namanya feat: add logic to course.ts function, nah kalo bisa commit-nya itu dikit-dikit agar commit message-nya juga lebih kegambar, jangan semua jadi baru di-commit, misalnya commit-nya per procedure feat: added getAllCourses procedure, feat: added getStudentsListOnCourseId procedure, feat: added insertNewCourse procedure, dll. atau dengan cara lainnya yang intinya commit-nya tuh lebih micro aja. 😄