-
Notifications
You must be signed in to change notification settings - Fork 5
Резюме зустрічі 10.07.15
romach edited this page Aug 2, 2015
·
2 revisions
- Існує можливіть конфліктів між двома послідовними пулл-реквестами, коли друга задача блокується першою (В першій розробляється функціонал для другої). Вирішення проблеми: створення об'єктів-заглушок;
- Інша причина конфлікту: редагування одного й тогож файлу декількома людьми; Рішення: рознесення конфігів по декілька файлів;
- Pull-request можна робити покроково. Умови: проходження всіх тестів, нема помилок компіляції, проект має бути у логічно завершеному стані. Внаслідок цього pull-request буде меншого розміру, легше буде проводити code-review.
- Необхідно робити колективний code-review pull-request'ів. В наслідок цього відбувається розподіл знань серед усіма учасниками проекту.
- Було піднято питання про відсутність java-doc коментарів в коді. Рішення: писати самодокументований, зрозумілий код. Корисна книга - "Clean code". По бажанню можна писати коментарі. Проблема за цього підходу: необхідність підтримання одночасно коду і коментарів. Також корисно писати юніт-тести.
- Кожна зміна в окремому файлі. Не можна змінювати файл change-set'а після публікації коду в master.