search

SemenMartynov / Software-Engineering-2022

Peter the Great St. Petersburg Polytechnic University: Software Engineering course 2022
MIT License
4 stars 1 forks source link

MoneyConverter-KZTtoUSD #237

Open kaamapulyaaa opened 1 year ago

kaamapulyaaa commented 1 year ago

Student, group: 3530901/90101 Курбанова К. URL: https://github.com/kaamapulyaaa/MoneyConverter-KZTtoUSD.git Git approach: git-flow

SemenMartynov commented 1 year ago
  1. Прямые коммиты в master/develop (a87058a, 73c7aec, 3e6a764)
  2. Ни одна feature/ ветка не верунлась в develop, но они каким-то диким образом мерджатся друг с другом
  3. Ни одна issue в репозитории не закрыта, и на один коммит не слинкован ни с одной issue
  4. Часть feature/ веток следует конвенции об именовании веток, часть -- нет
  5. ...ещё целая пачка прямых коммитов в master (это должно было быть запрещено политикам ещё в рамках первой работы)
  6. Docker контейнер валится на запуске: он не знает как запустить ноду, она там не установлена.
  7. Ключи API лучше передавать переменными окружения image