Основной посыл комментариев пулл-реквест #1 #python 1. Ст | < codereview />: IT карьера под ключ

Основной посыл комментариев

пулл-реквест #1
#python

1. Стукрута проекта
Лучше всего создать папку src, разбить логику по моделям, и сервисам и вынести все из корневой папки.

2. Доработать модели
Вынести BaseModel в отдельный файл. В Vacancy добавить employment type с EnumField.

3. Нейминг
Избавиться от Helpers, чтобы поднять читаемость кода.

коммит #2
#typescript #angular

1. Задать точный тип возврата функции
Функция calculateWinner вернет X | O | null.

2. Избегать повторяемость кода
X и O используется в нескольких местах. Лучше вынести в качестве констант.

3. Неявное приведение объекта к boolean
Вместо проверки if(Winner) => isWinnerExists, и вместо this.squares[idx] => используй includes.

4. Избегать логических ошибок
makeMove будет вызывать calculateWinner, даже если ничего не изменилось.

Хотел бы выделить следующих ревьюеров

fallncrlss - за информативный и структурированный комментарий для python.
QoreCode - за большой объем полезных комментариев для TypeScript